383898 - Swapped image from onmouseover is offset/positioned incorrectly

Status: VERIFIED FIXED

(Core :: Layout, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

Build ID: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9a6pre) Gecko/20070609 Minefield/3.0a6pre

Summary: Swapped image from onmouseover is offset/positioned incorrectly

Steps to Reproduce:

1. Load the URL and mouse over the "Search" image next to the textfield.
2. Look at its swapped image.

Expected Results:

The red "Search" image should replace the black one.

Actual Results:

The red "Search" image is offset quite a bit; in a 1024x768 window it loads halfway offscreen to the right.

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Files needed to run the testcase in the URL.

Comment 2

[Boris Zbarsky [:bzbarsky]]

What site does this come from?

In any case, this regressed between 2007-03-15-01 and 2007-03-16-01, almost certainly as a result of bug 111207.  Hard to tell with the huge gobs of JS involved: I can't even find the showMenu function in question in that mess.

Whatever we're doing here is not very compatible with IE; we probably need to fix that for 1.9 if it's going to break sites like this.

Comment 3

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #2)
> What site does this come from?

http://www.pozen.com (you should see the _inline_ code, if you think the JS is a mess :-( )
 
<snip> 

Comment 4

[Martijn Wargers (dead)]

More likely a regression from bug 372365, it's also in that regression range.

Comment 5

[Martijn Wargers (dead)]

Attachment: testcase

I didn't get this test completely by minimizing the original one, but I'm pretty sure it's the same issue (it has the same regression range).
You should see no red on the page.

Comment 6

[Boris Zbarsky [:bzbarsky]]

This might get fixed by the patch for bug 375003, then.

Comment 7

[Mats Palmgren (inactive)]

Should be working now in builds dated 2007-06-14 or newer.
The mochitests on bug 375003 covers the 2nd testcase (and a lot more).

-> FIXED (by bug 375003)

Comment 8

[Jeff Walden [:Waldo]]

in-testsuite+ per comment 7.

Comment 9

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED using Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9a6pre) Gecko/20070615 Minefield/3.0a6pre with all of the testcases in this bug (both live and attached).