Summary: | Web Inspector: Convert sidebar classes to ES6 | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Timothy Hatcher <timothy> | ||||||
Component: | Web Inspector | Assignee: | Timothy Hatcher <timothy> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | graouts, joepeck, jonowells, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 142891 | ||||||||
Attachments: |
|
Description
Timothy Hatcher
2015-03-26 11:44:40 PDT
Created attachment 249503 [details]
Patch
Created attachment 249504 [details]
Patch (Ignore Space)
Comment on attachment 249504 [details] Patch (Ignore Space) View in context: https://bugs.webkit.org/attachment.cgi?id=249504&action=review r=me > Source/WebInspectorUI/UserInterface/Views/ApplicationCacheDetailsSidebarPanel.js:140 > +WebInspector.ApplicationCacheDetailsSidebarPanel.StyleClassName = "application-cache"; For single use StyleClassName's I've inlining them. Donno if you feel the same, but this feels really verbose for little benefit. > Source/WebInspectorUI/UserInterface/Views/DOMNodeDetailsSidebarPanel.js:36 > - this.element.classList.add(WebInspector.DOMNodeDetailsSidebarPanel.StyleClassName); > + this.element.classList.add("dom-node"); I see sometimes you have been doing this as well. > Source/WebInspectorUI/UserInterface/Views/DOMNodeDetailsSidebarPanel.js:589 > + console.log(attribute.name, attribute.value); > + > var node = new WebInspector.DataGridNode({name: attribute.name, value: attribute.value || ""}, false); > + console.log(node); Looks like debug logging we should remove. > Source/WebInspectorUI/UserInterface/Views/Sidebar.js:274 > + CollapsedStateDidChange: "sidebar-sidebar-collapsed-state-did-change", Compared to the others, this one has an extra "sidebar-". Comment on attachment 249503 [details] Patch https://trac.webkit.org/r182041 |