86746 - layout flips back and forth between allowing space for a float or not [FLOAT] {ll} [INLINE] [TEXT]

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Scott Collins]

Go to

  http://ScottCollins.net/journal/#progress

note whether the section `work in progress' has incorrectly left room as if to
account for the `last modified' float above and to the right.  Grow the window.
 Layout fixed.  Grow the window.  Extra space again.  This is very easy to show
on the Mac because of bug #86740, but Waterson and I verified on all first tier
platforms.

Comment 1

[Scott Collins]

I'll try to find a minimal test case.  CC'ing hixie, maybe he can help.

Comment 2

[Scott Collins]

Attachment: a minimal test case

Comment 3

[Scott Collins]

as the attached minimal test case shows, key to this bug is |text-align: right|
on the float.  Playing with the font size and margins of the float to make it
interfere or not with the objects below it gives you interesting results.  The
real problem seems to be the floats decision to need a line break sometimes. 
Hope this helps.

Comment 4

[Hixie (not reading bugmail)]

The problem is actually that there is a space at the start and end of the 
float's contents, which is rather strange. See the following tests:
   http://www.hixie.ch/tests/adhoc/css/box/float/011.xml
   http://www.hixie.ch/tests/adhoc/css/box/float/012.xml
   http://www.hixie.ch/tests/adhoc/css/box/float/013.xml

Note. You will need the Ahem font for these tests. You may obtain a copy here:
   http://www.hixie.ch/resources/fonts/

Comment 5

[Scott Collins]

I've adjusted my journal page (intentionally) so it no longer exhibits the
problem.  Updating the URL link above to point to my test case (if you find that
helpful).  I downloaded Hixie's font and tried his test cases; they seem even
simpler than mine :-)

Comment 6

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

My patch on bug 85876 fixes the flipping.  I don't think it fixes Ian's test
cases, though.

Comment 7

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Taking this bug, since I've fixed the original bug.  I think the other problems
with Ian's testcases are part of bug 4248.

Ian - if you think your testcases show a bug that should be separate from bug
4248, please file it.

Comment 8

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Fix checked in 2001-07-18 19:33 PDT (see bug 85876 for more details).