[webkit-reviews] review denied: [Bug 170418] Web Inspector: Reorder Debugger tab sidebar panels: Scope Chain, Resource, Probes : [Attachment 306140] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Apr 3 19:58:05 PDT 2017
Timothy Hatcher <timothy at hatcher.name> has denied Matt Baker
<mattbaker at apple.com>'s request for review:
Bug 170418: Web Inspector: Reorder Debugger tab sidebar panels: Scope Chain,
Resource, Probes
https://bugs.webkit.org/show_bug.cgi?id=170418
Attachment 306140: Patch
https://bugs.webkit.org/attachment.cgi?id=306140&action=review
--- Comment #9 from Timothy Hatcher <timothy at hatcher.name> ---
Comment on attachment 306140
--> https://bugs.webkit.org/attachment.cgi?id=306140
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=306140&action=review
> Source/WebInspectorUI/UserInterface/Views/ContentBrowserTabContentView.js:154
> + WebInspector.detailsSidebar.addSidebarPanel(sidebarPanel,
(a, b) => {
Wouldn't we know the index it needs inserted at? You would be better off adding
a inserSidebarPanel function that takes an index. No need to sort the array for
one item insert when the order is known already.
> Source/WebInspectorUI/UserInterface/Views/Sidebar.js:63
> + addSidebarPanel(sidebarPanel, comparator)
The index implementation you add below can be used for a new
insertSidebarPanel. And you can make addSidebarPanel use it to reduce
duplication.
More information about the webkit-reviews
mailing list