<dd> <dt> tags dont work as expected

NEW
Unassigned

Status

()

Core
Layout: Floats
7 years ago
2 years ago

People

(Reporter: burak, Unassigned)

Tracking

({regression})

Trunk
regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0.1) Gecko/20100101 Firefox/4.0.1
Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0.1) Gecko/20100101 Firefox/4.0.1

In older version of firefox, this tags used to work as expected. with the newer version: <dd> tag that follows the 'Previous Topic:' section shown at the wrong section. 



Reproducible: Always


Actual Results:  
"Busted" link is located at unexpected location on the page. 

Expected Results:  
"Busted" link should be located next to "Previous Topic:" on the page.

Comment 1

7 years ago
Reproduced, somewhat. For me "Busted" is indeed located next to "Previous Topic:", but instead "Topic pages:" and "1" has been split.
Mozilla/5.0 (X11; Linux x86_64; rv:2.0.1) Gecko/20100101 Firefox/4.0.1
Mozilla/5.0 (X11; Linux x86_64; rv:6.0a1) Gecko/20110519 Firefox/6.0a1

WFM:
Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.2.17) Gecko/20110420 Firefox/3.6.17


Regression range:

Last good nightly: 2010-08-05 First bad nightly: 2010-08-06

Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=bbefb7bcb41e&tochange=26ee1b556bd9
OS: Windows 7 → All
Hardware: x86 → All
Version: unspecified → Trunk
(Reporter)

Comment 3

7 years ago
I dont think any of those validation errors have anything to do with this bug. Firefox 3.x version shows the css as expected, this happened version starting from 4.x

Comment 4

7 years ago
The first bad revision is:
changeset:   48999:66c78df18e50
user:        L. David Baron <dbaron@dbaron.org>
date:        Thu Aug 05 21:59:20 2010 -0700
summary:     Pass correct *remaining* height to float manager, rather than the whole height of the content area.  (Bug 563584, patch 22)  r=roc

Updated

7 years ago
Component: General → Layout
Product: Firefox → Core
QA Contact: general → layout

Updated

7 years ago
Keywords: regression
Hmm.  So this page is using a negative top margin to pull the floats up.  But it's using a "-2em" margin, and there's no good reason that would be enough to line the float up exactly with the <dd> to its left (even though the latter has line-height forced to 2em.  So as far as I can tell, the float just doesn't fit and so moves to the next column....
Blocks: 563584
A minimal testcase would be a good idea.
Keywords: testcase-wanted

Comment 7

7 years ago
Created attachment 534261 [details]
Minimal testcase, safe for work. :-)

I've written a minimal testcase that mimics the layout on the site and shows the layout error with Nightly 2010-08-06 and on.

Updated

7 years ago
Keywords: testcase-wanted

Comment 8

7 years ago
Mozilla/5.0 (Windows NT 5.1; rv:6.0a1) Gecko/20110503 Firefox/6.0a1

Considering the added testcase - setting resolution to New.
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 9

7 years ago
Regression window:

Last good:
Mozilla/5.0 (Windows NT 5.1; rv:2.0b4pre) Gecko/20100805 Minefield/4.0b4pre
http://hg.mozilla.org/mozilla-central/rev/bbefb7bcb41e


First bad:
Mozilla/5.0 (Windows NT 5.1; rv:2.0b4pre) Gecko/20100806 Minefield/4.0b4pre
http://hg.mozilla.org/mozilla-central/rev/26ee1b556bd9

Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=bbefb7bcb41e&tochange=26ee1b556bd9

Comment 10

7 years ago
(In reply to comment #9)
> Regression window:
...

And changeset 66c78df18e50 has already been tracked down in comment 4. :-)

Comment 11

7 years ago
And thank you for that :)
My guess is that either:
 * the new behavior is correct, or
 * the issue is that we no longer fit a 0-height (or negative height) float in a 0-height space
I can reproduce this issue on 49.0a1 Windows 8.1 x64.
Component: Layout → Layout: Floats
You need to log in before you can comment on or make changes to this bug.