116511 - element.offset* incorrect for relatively positioned elements inside table, td, and th

Status: RESOLVED FIXED

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

Description

[Christopher Aillon (sabbatical, not receiving bugmail)]

On AIM.com the menus in the top left are positioned badly.

I'm filing this as critical because it prevents me from downloading AIM at all
while using Mozilla 2001122108.  There seems to be no other way to get to the
downloads area without the menus.

I am assuming that they are using old offset code before ours got fixed, but
haven't looked at the code extensively.

For reference, see bug 90617 and bug 81290.

Comment 1

[Bob Clary [:bc] (inactive)]

This menu works in NS 6.2.1/RTM, and Mozilla 0.9.5 and 0.9.6 but not in recent
Mozilla or NS 6.2+ builds.

http://aim.aol.com/aimnew/newreg/aim_reg.css
http://aim.aol.com/aimnew/newreg/aim_reg.css
http://www.aim.com/chrome_data/model.js
http://www.aim.com/chrome_data/menu_config.js
http://www.aim.com/chrome_data/menu_lib.js
http://ads.web.aol.com/file/adsWrapper.js
http://ads.web.aol.com/file/adsEnd.js

Chris, this may be a browser bug and not evangelism

Comment 2

[basic]

DOM 0

Comment 3

[basic]

Attachment: testcase

Comment 4

[Johnny Stenback (:jst)]

Oh nice, the reason for this is that the .offsetParent of the DIV that's shown
is the BODY in IE, and in mozilla it's the closest TD, as it IMO should be in IE
too.

Does anyone have any ideas why the .offsetParent of the submenu DIV element is
the BODY in IE in this case?

Comment 5

[Johnny Stenback (:jst)]

Attachment: testcase that shows the offsetParent of the submenu

Comment 6

[Johnny Stenback (:jst)]

The reason the .offsetParent is the BODY in IE is that the DIV in question is
relatively positioned. Patch coming up.

Comment 7

[Johnny Stenback (:jst)]

Attachment: Proposed fix, and also eliminate some extra string copying

Comment 8

[Fabian Guisset]

Comment on attachment 62612 [details] [diff] [review]
Proposed fix, and also eliminate some extra string copying

r=fabian

Comment 9

[Johnny Stenback (:jst)]

*** Bug 117455 has been marked as a duplicate of this bug. ***

Comment 10

[Johnny Stenback (:jst)]

*** Bug 116028 has been marked as a duplicate of this bug. ***

Comment 11

[Sebastian Biallas]

Additional URLs to verify from Bug 116028:

http://www.cbsmarketwatch.com
http://home.netscape.com/ex/shak/shopping/index.html

Comment 12

[vidur (gone)]

Comment on attachment 62612 [details] [diff] [review]
Proposed fix, and also eliminate some extra string copying

sr=vidur

Comment 13

[Johnny Stenback (:jst)]

Fixed.