Closed Bug 2813 Opened 26 years ago Closed 26 years ago

"frame-change" style event not yet implemented

Categories

(Core Graveyard :: GFX, defect, P1)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: mikepinkerton, Assigned: peterl-retired)

Details

(I know this is on the schedule but since it is a dependency, peter wanted me to file a bug so he can track its progress). For toolbars to implement collapsing correctly, I need to be able to change the display type of a frame on the fly (eg, set something to display:none after it has been created). This currently isn't implemented.
OS: Mac System 8.5 → All
Hardware: Macintosh → All
changing platform/os to "All"
Assignee: troy → nisheeth
Setting all current Open/Normal to M4.
Assignee: nisheeth → peterl
The fix for this is checked in. Setting the display property on HRs and other leaf elements was working. Setting the same property on container elements like tables wasn't: the style context would not get re-resolved with the new value of the display attribute. Peter is currently taking a look at this. Am re-assigning the bug to him.
Am I correct in assuming that this "bug" that peter is looking into would prevent toolbars (which are block-frame containers) from correctly respecting display:none?
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
This seems to be working fine now...
QA Contact: 3819
I will mark verified assuming pinkerton agrees with fix.
Status: RESOLVED → REOPENED
This doesn't seem to work for me. When I set toolbars to have display: none, I get a reflow, but the toolbar frames are not destroyed and recreated (i put breakpoints in the constructor/destructor). You can still see the "collapsed" toolbar. Reopening
Resolution: FIXED → ---
Clearing fixed resolution
Status: REOPENED → RESOLVED
Closed: 26 years ago26 years ago
Resolution: --- → FIXED
Ok, closing this bug and reopening a different one. I wasn't aware that there were multiple stages going on here.
I can't verify this, however. Maybe nisheeth can.
Status: RESOLVED → VERIFIED
marking verified since another bug was opened
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.