Changeset 185669 in webkit

Timestamp:

Jun 17, 2015, 2:33:32 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

Web Inspector: Show/Hide sidebar buttons have inconsistent highlighted state
​https://bugs.webkit.org/show_bug.cgi?id=145073

Patch by Diego Pino Garcia <Diego Pino Garcia> on 2015-06-17
Reviewed by Joseph Pecoraro.

• UserInterface/Views/ContentBrowserTabContentView.js:

(WebInspector.ContentBrowserTabContentView.prototype._detailsSidebarPanelSelected):

Location:

trunk/Source/WebInspectorUI

Files:

• ChangeLog (modified) (1 diff)
• UserInterface/Views/ContentBrowserTabContentView.js (modified) (1 diff)

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2015-06-17  Diego Pino Garcia  <dpino@igalia.com>
+
+        Web Inspector: Show/Hide sidebar buttons have inconsistent highlighted state
+        https://bugs.webkit.org/show_bug.cgi?id=145073
+
+        Reviewed by Joseph Pecoraro.
+
+        * UserInterface/Views/ContentBrowserTabContentView.js:
+        (WebInspector.ContentBrowserTabContentView.prototype._detailsSidebarPanelSelected):
+
 2015-06-16  Joseph Pecoraro  <pecoraro@apple.com>
 

trunk/Source/WebInspectorUI/UserInterface/Views/ContentBrowserTabContentView.js

@@ -228 +228 @@
             return;
 
+        this._showDetailsSidebarItem.activated = !WebInspector.detailsSidebar.collapsed;
         this._showDetailsSidebarItem.enabled = WebInspector.detailsSidebar.sidebarPanels.length;