Closed
Bug 145969
Opened 22 years ago
Closed 22 years ago
Can't persist open state of content view trees
Categories
(Core :: XUL, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: neil, Assigned: hewitt)
Details
(Keywords: polish)
Attachments
(1 file, 1 obsolete file)
|
755 bytes,
patch
|
janv
:
review+
jag+mozilla
:
superreview+
|
Details | Diff | Splinter Review |
Because nsTreeContentView::ToggleOpenState removes the "open" attribute it won't persist properly.
|
Reporter | |
Comment 1•22 years ago
|
||
|
|
Reporter | |
Updated•22 years ago
|
|
||
Comment 2•22 years ago
|
||
r=varga on
- row->mContent->UnsetAttr(kNameSpaceID_None, nsXULAtoms::open, PR_TRUE);
+ row->mContent->SetAttr(kNameSpaceID_None, nsXULAtoms::open,
NS_LITERAL_STRING("false"), PR_TRUE);
|
|
Reporter | |
Comment 3•22 years ago
|
||
Oops, I'm suffering from multiple patch syndrome again :-(
Attachment #84434 -
Attachment is obsolete: true
|
|
||
Comment 4•22 years ago
|
||
Comment on attachment 84440 [details] [diff] [review] Proposed patch r=varga It would be nice to add a comment why we need to do that this way.
Attachment #84440 -
Flags: review+
|
|
Reporter | |
Comment 5•22 years ago
|
||
Jan, be sure to do that when you check it in for me :-)
|
||
Comment 6•22 years ago
|
||
Comment on attachment 84440 [details] [diff] [review] Proposed patch sr=jag
Attachment #84440 -
Flags: superreview+
|
|
||
Comment 7•22 years ago
|
||
checked in
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: shrir → xptoolkit.widgets
You need to log in
before you can comment on or make changes to this bug.
Description
•