Closed Bug 389378 Opened 17 years ago Closed 9 years ago

www.zdjecia.pl - page menu is totally broken

Categories

(Core :: Layout: Floats, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: lqMozilla, Unassigned)

References

(Blocks 1 open bug, )

Details

(Keywords: regression, testcase)

Attachments

(4 files)

User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7pre) Gecko/2007072304 Minefield/3.0a7pre Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7pre) Gecko/2007072304 Minefield/3.0a7pre Page menu is totally broken. Reproducible: Always Steps to Reproduce: 1.Open browser 2.Go to page www.zdjecia.pl Actual Results: Page menu is totally broken. See attached screenshots from device and desktop. Expected Results: Page menu under top image looks ok and is in one line.
What attached screenshots? :)
Version: unspecified → Trunk
Attached file Testcase (deleted) —
The bug occurs because a line break occurs after the space in one of the menu entries. I'm guessing this is caused by the recent line breaking fixes - bug 255990 ? Haven't looked for duplicates but I didn't see anything in the dependencies of bug 255990. I'll leave it to some one else to work out if it's expected that there be a line break after the space in this case.
Component: Menus → General
Keywords: testcase
QA Contact: menus → general
Component: General → Layout
Product: Firefox → Core
QA Contact: general → layout
this regressed even before bug 255990 was fixed btw, it was marked blocking bug 300030 ;)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking1.9?
Attached file Testcase #2 (deleted) —
Both testcases triggers this assertion in a debug build: ###!!! ASSERTION: If we asked for force-fit, it should have been placed: 'placed || !forceFit', file nsBlockReflowState.cpp, line 548
Component: Layout → Layout: Floats
OS: Windows XP → All
QA Contact: layout → layout.floats
Keywords: assertion
Testcase #1 and #2 is WFM due to bug 387201 and the assertion is gone too. I think there is still a bug here though.
Depends on: 387201
Flags: in-testsuite?
Attached file Testcase #3 (deleted) —
Is the new testcase a regression?
(In reply to comment #7) > Is the new testcase a regression? The reflow branch landing changed the layout. I think there are two issues here. The first is that we layout two floats on top of each other in Testcase #3. I think the layout should be more like #4, but I'm not really sure what -moz-float-edge:margin-box really means... The second issue is that we only make the outer float wide enough to fit the widest descendant. Firefox 2 makes the outer float wide enough to fit all floats on one line (given a wide enough window). I think Firefox 2 better implements shrink-to-fit according to CSS 2.1 10.3.5: http://www.w3.org/TR/CSS21/visudet.html#float-width FWIW, Fx 2 and IE7 have the same layout; Opera9, Webkit and Fx trunk have the same layout (for issue 2 that is).
Flags: blocking1.9? → blocking1.9-
Keywords: assertion
Blocks: 422378
Works for me with the following specifications: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:48.0) Gecko/20100101 Firefox/48.0 Though it may be possible that the website went through a redesign or two since the filing of this bug. If you can still reproduce the issue, please reopen and provide current information. Thanks.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: