Closed Bug 892425 Opened 11 years ago Closed 11 years ago

Defect - Layout in grid with bookmarks changes after navigation, also displays scrollbars

Categories

(Firefox for Metro Graveyard :: Firefox Start, defect, P2)

x86_64
Windows 8.1
defect

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 25

People

(Reporter: jimm, Assigned: sfoster)

References

Details

(Whiteboard: feature=defect c=tbd u=tbd p=3)

Attachments

(2 files, 1 obsolete file)

Attached image screenshot (deleted) —
Odd quirky bug, STR: 1) have eight bookmarks saved 2) open metrofx note that bookmarks are laid out in two columns, four bookmarks each. 3) navigate to a site using touch 4) navigate back using the overlay back button result: bookmarks are now laid out in a single column, eight bookmarks with the lower bookmark slightly clipped by the nav bar. 5) move the mouse result: scrollbars on the bookmarks column reproduced on a Surface Pro.
Summary: Layout in grid with bookmarks changes after navigation, also displays scrollbars → Defect - Layout in grid with bookmarks changes after navigation, also displays scrollbars
Whiteboard: feature=defect c=tbd u=tbd p=0
Priority: -- → P2
Assignee: nobody → sfoster
Blocks: metrov1it11
No longer blocks: metrov1defect&change
Status: NEW → ASSIGNED
QA Contact: jbecerra
Whiteboard: feature=defect c=tbd u=tbd p=0 → feature=defect c=tbd u=tbd p=3
Attached patch Stabilize meta-section grid sizing (obsolete) (deleted) — Splinter Review
The attached fixes the issue for me. Jimm can you confirm? I've left the outline on the bookmarks grid to make it more apparent. Also any feedback on the fix appreciated. Basically I leave space for a possible horizontal scrollbar.
Attachment #781159 - Flags: feedback?(mbrubeck)
Attachment #781159 - Flags: feedback?(jmathies)
Attachment #781159 - Flags: feedback?(mbrubeck) → feedback+
Comment on attachment 781159 [details] [diff] [review] Stabilize meta-section grid sizing fixes the problem for me. the grid layout remains unchanged between the nav and the nav back.
Attachment #781159 - Flags: feedback?(jmathies) → feedback+
Tightens up some of the richgrid sizing rules and logic - we now use the bound node as the reference bounding box if it has flex. By adding some margin to leave room for a horizontal scrollbar, the available space for the start grids doesn't change so we don't get grid rearranging as the precise/imprecise input value flips.
Attachment #781159 - Attachment is obsolete: true
Attachment #781258 - Flags: review?(mbrubeck)
Attachment #781258 - Attachment is patch: true
Attachment #781258 - Attachment mime type: message/rfc822 → text/plain
Attachment #781258 - Flags: review?(mbrubeck) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 25
User Agent: Mozilla/5.0 (Windows NT 6.2; Win64; x64; rv:26.0) Gecko/20100101 Firefox/26.0 Build ID: 20130806104538 Built from http://hg.mozilla.org/mozilla-central/rev/1e381c91885d WFM Tested on windows 8 using latest nightly. Followed steps provided in comment0 and got expected result.
Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0 Build ID: 20130815030203 Built from http://hg.mozilla.org/mozilla-central/rev/a8daa428ccbc WFM Tested on windows 8 using latest nightly for iteration-12. I don't see any issue after using overlay back button.
Status: RESOLVED → VERIFIED
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: