Closed
Bug 829416
Opened 12 years ago
Closed 12 years ago
[Social API] ambient popup is displayed at center of screen when toolbar is in Text mode
Categories
(Firefox Graveyard :: SocialAPI, defect)
Firefox Graveyard
SocialAPI
Tracking
(firefox18 unaffected, firefox19 unaffected, firefox20 verified, firefox21 verified)
VERIFIED
FIXED
Firefox 21
Tracking | Status | |
---|---|---|
firefox18 | --- | unaffected |
firefox19 | --- | unaffected |
firefox20 | --- | verified |
firefox21 | --- | verified |
People
(Reporter: andyjpn, Assigned: jaws)
References
Details
(Whiteboard: [testday-20130301])
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
mixedpuppy
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_7_4) AppleWebKit/537.11 (KHTML, like Gecko) Chrome/23.0.1271.97 Safari/537.11
Steps to reproduce:
1. Right-click on the toolbar
2. Select "Customize..."
3. Select the "Text" item from the Show: combobox
4. Click "Done"
5. Click "See all friend requests" item in the toolbar
Actual results:
A dialog which shows all friend requests is displayed at the center of the toolbar.
Expected results:
The dialog should be displayed below "See all friend requests" item in the toolbar.
Updated•12 years ago
|
Component: Untriaged → SocialAPI: Providers
Comment 1•12 years ago
|
||
This may be a side effect of bug 811835 (whose other symptoms are visible in the screenshot).
Component: SocialAPI: Providers → SocialAPI
Comment 2•12 years ago
|
||
Oh, though actually that looks like post-fix. So maybe there are still issues here related to popup positioning.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → jaws
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
OS: Mac OS X → All
Hardware: x86 → All
Assignee | ||
Comment 3•12 years ago
|
||
Attachment #702885 -
Flags: review?(mixedpuppy)
Attachment #702885 -
Flags: review?(felipc)
Assignee | ||
Comment 4•12 years ago
|
||
Firefox 18 and 19 should be unaffected because they don't use the standard toolbarbutton approach. Firefox 20 is affected because the patch that introduced the standard toolbarbutton approach got uplifted to Fx20Aurora.
status-firefox18:
--- → unaffected
status-firefox19:
--- → unaffected
status-firefox20:
--- → affected
status-firefox21:
--- → affected
Depends on: 811835
Updated•12 years ago
|
Attachment #702885 -
Flags: review?(mixedpuppy) → review+
Assignee | ||
Updated•12 years ago
|
Attachment #702885 -
Flags: review?(felipc)
Assignee | ||
Comment 5•12 years ago
|
||
Assignee | ||
Comment 6•12 years ago
|
||
Comment on attachment 702885 [details] [diff] [review]
Patch
[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 811835
User impact if declined: misaligned popup panels for the SocialAPI
Testing completed (on m-c, etc.): locally, just landed on m-i
Risk to taking this patch (and alternatives if risky): low risk patch, very isolated
String or UUID changes made by this patch: none
Attachment #702885 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•12 years ago
|
Summary: [Social API] a dialog is poped up at wrong position → [Social API] ambient popup is displayed at center of screen when toolbar is in Text mode
Comment 7•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 21
Comment 8•12 years ago
|
||
Comment on attachment 702885 [details] [diff] [review]
Patch
low risk, approving for aurora.
Attachment #702885 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 9•12 years ago
|
||
Comment 10•12 years ago
|
||
Verified fixed on Firefox 20.0 beta 2 and Windows 7 64 bit.
I followed the reportes's steps to reproduce:
1. Right-click on the toolbar
2. Select "Customize..."
3. Select the "Text" item from the Show: combobox
4. Click "Done"
5. Click "See all friend requests" item in the toolbar
I could see the expected results:
The dialogue was displayed below "See all friend requests" item in the toolbar.
Whiteboard: [testday-20130301]
Comment 11•12 years ago
|
||
Thank you Gabriela. Could you please also test the latest Firefox 21.0a2 Aurora build?
Comment 12•12 years ago
|
||
Anthony, I just did. I verified fixed with the latest Aurora build too.
Comment 13•12 years ago
|
||
(In reply to Gabriela from comment #12)
> I verified fixed with the latest Aurora build too.
Thank you.
Updated•6 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•