273392 - Blocks jump when clicking on links

Status: VERIFIED FIXED

(Core :: Layout, defect)

Description

[Robert Kaiser]

On http://www.fynf.at/ I noticed all kinds of block elements (fieldsets, tables,
divs) in the middle part of the pages jumping in their widths when I'm clicking
on links inside them.

This actually looks kind of funny, and I managed to derive a very simplified
testcase for that issue that I'll attach in a minute.

This started happening quite early in the 1.8a series, I didn't see it happen
with 1.7x or earlier builds.

Comment 1

[Robert Kaiser]

Attachment: testcase 1: click on the links to see div widths jumping

First testcase:
Click on the links inside the bordered divs to see the widths jumping around.
If focus is on one of the links, you may even just click on the empty area of a
div to see such jumps...

Comment 2

[Robert Kaiser]

Attachment: testcase 2: a bit more complex, but the same thing basically

Second testcase:
This was made one step earlier when deriving the simplified testcase, it has
some more block-level elements so you just see more boxes jumping around ;-)

Comment 3

[Robert Kaiser]

I think this has something to do with overflow:auto; and the padding on the main
div, as it only happens when I have those two rules in the CSS.

CCing bz, roc and dbaron, as I think they may know what's going on with overflow...
This should probably go into a specialized layout component but I couldn't
figure out which...

Comment 4

[Robert Kaiser]

OK, I just checked and, like I remembered, this is a regression - it works
perfectly without any jumping on 1.7.x (tested with 1.7.3 right now).

Comment 5

[Robert O'Callahan (:roc) (email my personal email if necessary)]

No doubt related to one of the bugs blocking 261196

Comment 6

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Testcase #1 works for me on trunk.

Testcase #2 is fixed by my upcoming patch in bug 240276.

Comment 7

[Robert Kaiser]

(In reply to comment #6)
> Testcase #1 works for me on trunk.

I still see it jumping on a current trunk build when clicking the different
links there.

> Testcase #2 is fixed by my upcoming patch in bug 240276.

I'll wait for this one before shouting again though ;-)

(BTW, http://www.fynf.at/ from comment 0 now uses a different design by default
that doesn't show the problem. You can call the site via
http://www.fynf.at/?vd=3 with the old design though to still see that)

Comment 8

[Steve England [:stevee]]

Testcase fails for me with Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US;
rv:1.8b2) Gecko/20050422 Firefox/1.0+
Testcase works for me with Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US;
rv:1.8b2) Gecko/20050428 Firefox/1.0+
Resolved Fixed by bug 240276

Comment 9

[Robert Kaiser]

I see the pages in the testcases now getting wider than the screen and having a
vertical scrollbar, which is a bug as well, caused by that checkin.

The jumping is gone though with the checkin.

Comment 10

[Boris Zbarsky [:bzbarsky]]

kairo, is that bug filed?

Comment 11

[Robert Kaiser]

(In reply to comment #10)
> kairo, is that bug filed?

Not yet, as I'm not sure if I should file a different bug for the test cases in
here still not working correctly or if I should reopen that one, I thiught roc
might know.

(and no need to CC me on a bug I filed)

Comment 12

[Boris Zbarsky [:bzbarsky]]

I'd file a new bug and link to the testcases if the problem shown by the
testcases changed.  And I didn't notice you were the reporter; sorry about that...

Comment 13

[Robert Kaiser]

(In reply to comment #12)
> I'd file a new bug and link to the testcases if the problem shown by the
> testcases changed.

done, it's bug 292371 now.

Comment 14

[Steffen Wilberg]

Attachment: testcase 3

I still see this bug with the attached testcase. (This is a reduced testcase of
the forums on http://www.heise.de/newsticker/foren/go.shtml).

Comment 15

[Steffen Wilberg]

I added some stuff to make it validate as HTML 4.01 Strict.
Sufficient to show the bug is this:

<table>
  <td><a href="#">Click me!</a>&nbsp;</td>
</table>

Reopening.

Comment 16

[Boris Zbarsky [:bzbarsky]]

Steffen, same symptoms don't necessarily mean same bug.  If the site in the URL
field is fixed but you're seeing a similar problem on a different site, file a
new bug, please (and restore this bug to the state it was in).

Comment 17

[Steffen Wilberg]

OK, I filed bug 292534. Marking fixed again.

Comment 18

[Robert Kaiser]

verfied fixed on Mozilla/5.0 (X11; U; Linux i686; de-AT; rv:1.8b2) Gecko/20050505