158354 - {inc} layout changes on hover (works correctly in 0.9.4)

Status: RESOLVED DUPLICATE of bug 157444

(Core :: Layout: Floats, defect, P2)

Description

[Doron Rosenberg (IBM)]

Mouse over the links/images in the page and watch the layout change.  Removing
the font-weight:bold fixes this.

Have a minmal testcase, going to attach.

Comment 1

[Doron Rosenberg (IBM)]

Attachment: testcase

adding reduced testcase

Comment 2

[Christopher Hoess (gone)]

Er. Why shouldn't it change?
http://www.w3.org/TR/REC-CSS2/selector.html#dynamic-pseudo-classes suggests to
me that if we support changing the font-size (or font-weight) on :hover, that
reflowing the document is to be expected...

Comment 3

[Boris Zbarsky [:bzbarsky]]

Doron, bold text is wider than the same non-bold test.  The layout change is
correct.

Comment 4

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

Agreed, based on testcase.  (I get an error page for the main URL.)

Comment 5

[Doron Rosenberg (IBM)]

OOps, the testcase does not show the actual issue I was seeing, which I believe
is a bug.

Try http://www.opitec.com/CGI/IOIAHS_H or the new testcase:

http://www.nexgenmedia.net/evang/opitec.com/IOIAHS_H.html

Mousing over the image (which is outside the <a>) causes indenting.

Comment 6

[Boris Zbarsky [:bzbarsky]]

The bold has nothing to do with this.  Any style that causes an incremental
reflow on hover will trigger this problem (eg changing font-size or something
like that).

The <br clear="all"> seems to not be doing what it should; the lower floater is
getting repositioned to be to the right of the upper floater instead of below it...

Comment 7

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

Note that the <br clear="all"> is only behaving wrong when the image loads come
in later and we fill in a placeholder.  On reload, the initial display is fine.
 Could be a problem somewhere near nsBlockFrame::RememberReflowDamage or
PropagateReflowDamage, but it could be elsewhere.

Comment 8

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

Attachment: simplified testcase

Comment 9

[Madhur Bhatia]

happens on linux, too

Comment 10

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

There are a lot of float incremental reflow bugs lately...

Comment 11

[Hixie (not reading bugmail)]

This is a dup of another <br clear=all> bug filed recently.

Comment 12

[Hixie (not reading bugmail)]

...namely bug 157444.

Comment 13

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

*** This bug has been marked as a duplicate of 157444 ***