133233 - [regression] Dynamic menus positioned wrongly

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Markus Hübner]

Just move over the menu-items (Unternehmen, success stories) and the sub-menus
will appear, unfortunately not below the menues.

This regression started very recently - build 2002031903 is broken.
build 2002031803 is fine.
[testing on win-xp]

Comment 1

[Tuukka Tolvanen (sp3000)]

linux:
2002-03-20-06-trunk 2002032006 ok
2002-03-20-08-trunk 2002032008 menus are offset 1 menu label character right.

Also, with the 06 build, the page is reloaded (?... the progress indicator moves
anyhow) and laid out again when I resize the window, and the menus follow. With
later builds resizing window does seem to reload something too, but the layout
doesn't get properly rearranged, so the menus and stuff get qite out of place
relative to each other. Nothing on js console in either case.

Comment 2

[Arthur]

On linux 2002032421 I don't see mouse-over menus at all. Mozilla does reload the
page after resizing here too.

Comment 3

[Tuukka Tolvanen (sp3000)]

hrm, right, you need to first click on one of the words in the middle to get the
menus

Comment 4

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

I suspect dbaron's RuleNode checkin. DOM Inspector shows that the first cell of
the first table contains an IMG element with a width attribute set (with my
window width, the IMG width is 178) but the table cell is sized to (0,0).
Perhaps the IMG width attribute is not being reflected into style correctly?

Comment 5

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

Is this bug only about the menus being a character or so to the right of
directly below the menus, or is the mis-positioning more serious than that on
Windows?  Why do the dates of the regression in comment 0 and comment 1 disagree?

I doubt my rule node changes on the 18th would have caused anything like this,
unless it was the patch for bug 131454, in which case I'd suspect we would be
more correct now.

Comment 6

[Markus Hübner]

Attachment: Screenshot on win32

Comment 7

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

The amount the menus are off depends on the window width.

The page is trying to center the 775px-wide menubar GIF ("Das Unternehmen ...")
by putting it into a table with two cells, the first cell containing a spacer
GIF and the second cell containing the image. The space GIF's width attribute is
set by script, but somehow its table cell gets reflowed to width zero, so the
menu bar is left-aligned rather than centered. The menus are still centered,
however, so they appear to the right of their menu titles (assuming your window
is sufficiently wide).

Comment 8

[Markus Hübner]

This seems to have been fixed with bug 137428.
using build 2002042403 it's fine now :)