Assignee: general → general
QA Contact: pschwartau → ian
Created attachment 169661 [details] Testcase from reporter I'm seeing the bug, using: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a6) Gecko/20041224 Firefox/1.0+
Replacing the fieldset with a div fixes the testcase. The incr. reflow for the fieldset is somehow broken. It does not properly invalidate the upper border neither does it paint the content at the right place. Please note that one can click the link just at the lower edge of that link. Loading the testcase into the DOM inspector reveals where mozilla thinks the parts should be.
Assignee: general → nobody
Status: UNCONFIRMED → NEW
Component: DOM: Level 0 → Layout: Form Controls
Ever confirmed: true
QA Contact: ian → core.layout.form-controls
This causes a crash using the testcase: 1. load testcase and click on the "New file" link 2. resize the window wildly - make it very narrow and then wider again. Taking bug - I have a patch that fixes this and it also fixes the assertion in bug 70648 comment 20.
Assignee: nobody → mats.palmgren
Severity: minor → critical
Created attachment 170075 [details] [diff] [review] Patch rev. 1 There were many bugs in there... (make sure you have bug 236071 before you run this patch)
Comment on attachment 170075 [details] [diff] [review] Patch rev. 1 > nsFieldSetFrame::AppendFrames(nsPresContext* aPresContext, > nsFieldSetFrame::InsertFrames(nsPresContext* aPresContext, I think the duplicated code here should be factored out somehow. All the methods operating on mLegendFrame when munging frame lists probably need to assert that aListName is null, no? Other than those, looks fine. r=bzbarsky
Attachment #170075 - Flags: review?(bzbarsky) → review+
14 years ago
Created attachment 171427 [details] [diff] [review] Patch rev. 2 Addressing Boris' comment 8: Added new method to share the common legend code in Insert/AppendFrames. Assert that aListName is null when manipulating 'mFrames'. Also, return the result of 'mContentFrame->AppendFrames()'.
Comment on attachment 171427 [details] [diff] [review] Patch rev. 2 rubber-stamp sr=dbaron
Attachment #171427 - Flags: superreview?(dbaron) → superreview+
Checked in 2005-01-24 16:06 PDT. -> FIXED
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
*** Bug 280020 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.