Changeset 212578 in webkit

Timestamp:

Feb 17, 2017 2:39:45 PM (7 years ago)

Author:

BJ Burg

Message:

Web Inspector: Loc: Zoom level in Settings tab needs to use localized formatting
​https://bugs.webkit.org/show_bug.cgi?id=168465

Reviewed by Matt Baker.

• UserInterface/Views/SettingsTabContentView.js:

(WebInspector.SettingsTabContentView.prototype.layout):
(WebInspector.SettingsTabContentView):

Location:

trunk/Source/WebInspectorUI

Files:

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

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2017-02-17  Brian Burg  <bburg@apple.com>
+
+        Web Inspector: Loc: Zoom level in Settings tab needs to use localized formatting
+        https://bugs.webkit.org/show_bug.cgi?id=168465
+
+        Reviewed by Matt Baker.
+
+        * UserInterface/Views/SettingsTabContentView.js:
+        (WebInspector.SettingsTabContentView.prototype.layout):
+        (WebInspector.SettingsTabContentView):
+
 2017-02-17  Brian Burg  <bburg@apple.com>
 

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

@@ -185 +185 @@
                 let option = select.createChild("option");
                 option.value = level;
-                option.textContent = `${Math.round(level * 100)}%`;
+                option.textContent = Number.percentageString(level, 0);
                 option.selected = currentZoom === level;
             });