Closed Bug 717819 Opened 13 years ago Closed 13 years ago

position:absolute element inside a table reports wrong offsetParent

Categories

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

10 Branch
x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla12

People

(Reporter: t.brain, Assigned: ehsan.akhgari)

Details

Attachments

(1 file, 1 obsolete file)

User Agent: Mozilla/5.0 (compatible; MSIE 9.0; Windows NT 6.1; WOW64; Trident/5.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; .NET4.0C; .NET4.0E) Steps to reproduce: Per bug 10209, these elements now use the right parent for positioning, but offsetParent still reports the outer block element.
Attached patch Patch (v1) (obsolete) — Splinter Review
Assignee: nobody → ehsan
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #588456 - Flags: review?(bzbarsky)
Comment on attachment 588456 [details] [diff] [review] Patch (v1) Oh, ok, then! r=me
Attachment #588456 - Flags: review?(bzbarsky) → review+
Component: Untriaged → General
Product: Firefox → Core
QA Contact: untriaged → general
Component: General → DOM: Core & HTML
QA Contact: general → general
Attached patch Patch (v2)Splinter Review
Oops, we actually do need that code if we're dealing with table frames.
Attachment #588456 - Attachment is obsolete: true
Attachment #588557 - Flags: review?(bzbarsky)
Comment on attachment 588557 [details] [diff] [review] Patch (v2) I assume one of those tests (or some other test) failed when the block was removed completely? r=me if so
Attachment #588557 - Flags: review?(bzbarsky) → review+
Try run for d701fd1f8bbe is complete. Detailed breakdown of the results available here: https://tbpl.mozilla.org/?tree=Try&rev=d701fd1f8bbe Results (out of 234 total builds): success: 173 warnings: 41 failure: 20 Builds (or logs if builds failed) available at: http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/eakhgari@mozilla.com-d701fd1f8bbe
(In reply to Boris Zbarsky (:bz) from comment #4) > Comment on attachment 588557 [details] [diff] [review] > Patch (v2) > > I assume one of those tests (or some other test) failed when the block was > removed completely? r=me if so Yes, they did.
Flags: in-testsuite+
Target Milestone: --- → mozilla12
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: