82313 - Reflow not correctly positioning floats [FLOAT]

Status: RESOLVED FIXED

(Core :: Layout, defect, P2)

Description

[Hixie (not reading bugmail)]

STEPS TO REPRODUCE
   1. Open http://bugzilla.mozilla.org/showattachment.cgi?attach_id=19134
   2. Resize the window

EXPECTED RESULTS
   As the window is resized, the floats should wrap around.

ACTUAL RESULTS
   The rendering is correct on the initial reflow, but subsequent reflows don't
   get the right layout.

NOTES
   The original testcase:
      http://www.geocities.com/theamigo42/mozfloatbug.html
   does this right for some reason.

Comment 1

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

This seems OK in current builds.

Comment 2

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

Er, now I'm looking at it on Windows and it doesn't.  We seem to be wrapping the
text and not wrapping the floats.

Comment 3

[Josh Harding]

Is this really a bug?

The main difference I see is that the attachment
<http://bugzilla.mozilla.org/showattachment.cgi?attach_id=19134> has the code:

 body {
   white-space: nowrap
 }

which is not present in my test case:
<http://www.geocities.com/theamigo42/mozfloatbug.html>

Isn't that what white-space: nowrap is supposed to do?

Comment 4

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

no, 'white-space: nowrap' should not affect floats.  But I think I know why it's
happening.  See bug 95511 / bug 95512.

Comment 5

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

See also bug 97383, although the comment I made before doesn't make as much
sense to me now as when I wrote it.

Comment 6

[Christopher Hoess (gone)]

WFM, fresh Linux CVS.

Comment 7

[Kevin McCluskey (gone)]

Reassigning to Alex

Comment 8

[Alexandru Savulov]

yeah it looks ok to me too. however there is another issue here: resizing the
window and hiting refresh casues elements to change the layout (test: enlarge
the window as far as possible/hit refresh, shrink the window to under width of
the widest floater, hit refresh)

need another bug on that

Comment 9

[Greg K.]

WorksForMe using FizzillaCFM/2002070913. All three boxes remain in the same
position regardless of browser width and reloading.

Comment 10

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

...which is an even more serious bug than the one here.

Comment 11

[Hixie (not reading bugmail)]

For me, CVS tip on a Linux-kernel OS shows exactly the same behaviour as I
originally reported. Window resizes have no effect, reloads do.

Comment 12

[Alexandru Savulov]

gets fixed by patch to bug 78820

Comment 13

[Alexandru Savulov]

got fixed by the fix for bug 78820