Closed
Bug 641894
Opened 14 years ago
Closed 14 years ago
Site menu is wrong size first time it is shown
Categories
(Firefox for Android Graveyard :: General, defect)
Firefox for Android Graveyard
General
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: wesj, Assigned: wesj)
Details
(Whiteboard: [fennec-checkin-postrc])
Attachments
(1 file)
|
1.19 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
The site menu is showing up far to large (tall) for me the first time it is shown. I think this is because we first show and anchor the popup, which sets a min-size on it in arrowbox.xml, before we hide the page actions that we don't want to show.
Patch moves the initialization so that the popup is set up before it is shown.
|
Assignee | |
Updated•14 years ago
|
Attachment #519453 -
Flags: review?(mark.finkle)
|
||
Comment 1•14 years ago
|
||
Comment on attachment 519453 [details] [diff] [review]
Patch
you need to verify that this change doesn't break the initial position of the site menu and verify that orientation/rotation changes with the site menu open still works OK.
Attachment #519453 -
Flags: review?(mark.finkle) → review+
|
|
Assignee | |
Updated•14 years ago
|
Assignee: nobody → wjohnston
|
||
Updated•14 years ago
|
Whiteboard: [fennec-checkin-postrc]
|
|
Assignee | |
Comment 2•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
||
Comment 3•14 years ago
|
||
VERIFIED FIXED on:
Build ID: Mozilla /5.0 (Android;Linux armv7l;rv:6.0a1) Gecko/20110524 Firefox/6.0a1 Fennec/6.0a1
Device: HTC Desire Z (Android 2.2)
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•