* SUMMARY Add a hidden property to TreeOutline. Currently it's stored in the DOM only, in the form of a CSS class.
<rdar://problem/23809742>
Created attachment 266940 [details] [Patch] Proposed Fix
Comment on attachment 266940 [details] [Patch] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=266940&action=review > Source/WebInspectorUI/UserInterface/Views/TreeOutline.js:68 > + this.element.classList.toggle("hidden", this._hidden); We can actually switch to using the "hidden" property on DOM elements: this.hidden = this._hidden;
(In reply to comment #3) > Comment on attachment 266940 [details] > [Patch] Proposed Fix > > View in context: > https://bugs.webkit.org/attachment.cgi?id=266940&action=review > > > Source/WebInspectorUI/UserInterface/Views/TreeOutline.js:68 > > + this.element.classList.toggle("hidden", this._hidden); > > We can actually switch to using the "hidden" property on DOM elements: > this.hidden = this._hidden; Neat, I'll change it and file another bug for doing a find/replace on our other uses of "hidden".
Created attachment 266944 [details] [Patch] Proposed Fix
Comment on attachment 266944 [details] [Patch] Proposed Fix Clearing flags on attachment: 266944 Committed r193791: <http://trac.webkit.org/changeset/193791>
All reviewed patches have been landed. Closing bug.