Open Bug 428480 Opened 16 years ago Updated 2 years ago

nsSplitterFrame should use internal state rather than attribute to track "collapsed" state

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Type:
defect

Tracking

()

Status:
REOPENED

People

(Reporter: sicking, Unassigned)

Details

From bug 423355 comment 70:

> That aside, the nsSplitterFrame changes look safe in any case, although it
> would make more sense for the splitter to just have an internal collapsed state
> that it can set and pick up during layout, avoiding the need for it to set its
> own "collapsed" attribute. File a bug for that?

This is in regard to SetAttr call for the collapsed attribute happening in nsSplitterFrameInner::UpdateState
This was fixed by the patch in bug 423355
Depends on: 423355
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
I'm confused. Which patch in bug 423355 addressed this issue?
Doh! Sorry, wrong bug
Status: RESOLVED → REOPENED
No longer depends on: 423355
Resolution: FIXED → ---
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.