Web Inspector: Show type info icon is blurry

https://bugs.webkit.org/show_bug.cgi?id=137125 Reviewed by Darin Adler. * UserInterface/Images/NavigationItemTypes.svg: * UserInterface/Views/ScriptContentView.js: (WebInspector.ScriptContentView): * UserInterface/Views/TextContentView.js: (WebInspector.TextContentView): * UserInterface/Views/TextResourceContentView.js: (WebInspector.TextResourceContentView): Canonical link: https://commits.webkit.org/154949@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@173996 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebKit · Sep 26, 2014 · 80fb283 · 80fb283
1 parent 59a8b38
commit 80fb283
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 8 deletions.
diff --git a/Source/WebInspectorUI/ChangeLog b/Source/WebInspectorUI/ChangeLog
@@ -1,3 +1,18 @@
+2014-09-25  Timothy Hatcher  <timothy@apple.com>
+
+        Web Inspector: Show type info icon is blurry
+        https://bugs.webkit.org/show_bug.cgi?id=137125
+
+        Reviewed by Darin Adler.
+
+        * UserInterface/Images/NavigationItemTypes.svg:
+        * UserInterface/Views/ScriptContentView.js:
+        (WebInspector.ScriptContentView):
+        * UserInterface/Views/TextContentView.js:
+        (WebInspector.TextContentView):
+        * UserInterface/Views/TextResourceContentView.js:
+        (WebInspector.TextResourceContentView):
+
 2014-09-25  Saam Barati  <saambarati1@gmail.com>
 
         Web Inspector: console.assert(bitString) TypeSet:50 

diff --git a/Source/WebInspectorUI/UserInterface/Images/NavigationItemTypes.svg b/Source/WebInspectorUI/UserInterface/Images/NavigationItemTypes.svg
diff --git a/Source/WebInspectorUI/UserInterface/Views/ScriptContentView.js b/Source/WebInspectorUI/UserInterface/Views/ScriptContentView.js
@@ -60,10 +60,9 @@ WebInspector.ScriptContentView = function(script)
     this._prettyPrintButtonNavigationItem.addEventListener(WebInspector.ButtonNavigationItem.Event.Clicked, this._togglePrettyPrint, this);
     this._prettyPrintButtonNavigationItem.enabled = false; // Enabled when the text editor is populated with content.
 
-    var showTypesImageSize = WebInspector.Platform.isLegacyMacOS ? 15 : 16;
     var toolTipTypes = WebInspector.UIString("Show type information");
     var activatedToolTipTypes = WebInspector.UIString("Hide type information");
-    this._showTypesButtonNavigationItem = new WebInspector.ActivateButtonNavigationItem("show-types", toolTipTypes, activatedToolTipTypes, "Images/NavigationItemTypes.svg", showTypesImageSize, showTypesImageSize);
+    this._showTypesButtonNavigationItem = new WebInspector.ActivateButtonNavigationItem("show-types", toolTipTypes, activatedToolTipTypes, "Images/NavigationItemTypes.svg", 13, 14);
     this._showTypesButtonNavigationItem.addEventListener(WebInspector.ButtonNavigationItem.Event.Clicked, this._toggleTypeAnnotations, this);
     this._showTypesButtonNavigationItem.enabled = false;
 

diff --git a/Source/WebInspectorUI/UserInterface/Views/TextContentView.js b/Source/WebInspectorUI/UserInterface/Views/TextContentView.js
@@ -51,10 +51,9 @@ WebInspector.TextContentView = function(string, mimeType)
     this._prettyPrintButtonNavigationItem.addEventListener(WebInspector.ButtonNavigationItem.Event.Clicked, this._togglePrettyPrint, this);
     this._prettyPrintButtonNavigationItem.enabled = this._textEditor.canBeFormatted();
 
-    var showTypesImageSize = WebInspector.Platform.isLegacyMacOS ? 15 : 16;
     var toolTipTypes = WebInspector.UIString("Show type information");
     var activatedToolTipTypes = WebInspector.UIString("Hide type information");
-    this._showTypesButtonNavigationItem = new WebInspector.ActivateButtonNavigationItem("show-types", toolTipTypes, activatedToolTipTypes, "Images/NavigationItemTypes.svg", showTypesImageSize, showTypesImageSize);
+    this._showTypesButtonNavigationItem = new WebInspector.ActivateButtonNavigationItem("show-types", toolTipTypes, activatedToolTipTypes, "Images/NavigationItemTypes.svg", 13, 14);
     this._showTypesButtonNavigationItem.enabled = false;
 };
 

diff --git a/Source/WebInspectorUI/UserInterface/Views/TextResourceContentView.js b/Source/WebInspectorUI/UserInterface/Views/TextResourceContentView.js
@@ -52,10 +52,9 @@ WebInspector.TextResourceContentView = function(resource)
     this._prettyPrintButtonNavigationItem.addEventListener(WebInspector.ButtonNavigationItem.Event.Clicked, this._togglePrettyPrint, this);
     this._prettyPrintButtonNavigationItem.enabled = false; // Enabled when the text editor is populated with content.
 
-    var showTypesImageSize = WebInspector.Platform.isLegacyMacOS ? 15 : 16;
     var toolTipTypes = WebInspector.UIString("Show type information");
     var activatedToolTipTypes = WebInspector.UIString("Hide type information");
-    this._showTypesButtonNavigationItem = new WebInspector.ActivateButtonNavigationItem("show-types", toolTipTypes, activatedToolTipTypes, "Images/NavigationItemTypes.svg", showTypesImageSize, showTypesImageSize);
+    this._showTypesButtonNavigationItem = new WebInspector.ActivateButtonNavigationItem("show-types", toolTipTypes, activatedToolTipTypes, "Images/NavigationItemTypes.svg", 13, 14);
     this._showTypesButtonNavigationItem.addEventListener(WebInspector.ButtonNavigationItem.Event.Clicked, this._toggleTypeAnnotations, this);
     this._showTypesButtonNavigationItem.enabled = false;