v3 UI should use four sig-figs to label y-axis of the main charts
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 6 Jan 2016 20:27:44 +0000 (20:27 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 6 Jan 2016 20:27:44 +0000 (20:27 +0000)
https://bugs.webkit.org/show_bug.cgi?id=152779

Reviewed by Antti Koivisto.

Increase the number of significant figures used in the main charts to four as done in v2 UI.

* public/v3/pages/chart-pane.js:
(ChartPane.constructor): Create a formatter with four significant figures.
* public/v3/pages/page-with-charts.js:
(PageWithCharts.mainChartOptions): Increase the width of y-axis labels.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@194652 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/v3/pages/chart-pane.js
Websites/perf.webkit.org/public/v3/pages/page-with-charts.js

index 3ae3c5aa35baf91d3496f4b013d75c3429ef75e3..448388be399d68c7303a2ad8419cebf1e36c3a75 100644 (file)
@@ -1,3 +1,17 @@
+2016-01-06  Ryosuke Niwa  <rniwa@webkit.org>
+
+        v3 UI should use four sig-figs to label y-axis of the main charts
+        https://bugs.webkit.org/show_bug.cgi?id=152779
+
+        Reviewed by Antti Koivisto.
+
+        Increase the number of significant figures used in the main charts to four as done in v2 UI.
+
+        * public/v3/pages/chart-pane.js:
+        (ChartPane.constructor): Create a formatter with four significant figures.
+        * public/v3/pages/page-with-charts.js:
+        (PageWithCharts.mainChartOptions): Increase the width of y-axis labels.
+
 2016-01-05  Ryosuke Niwa  <rniwa@webkit.org>
 
         v3 UI's time range slider is harder to use than that of v2 UI
index 5a1ef727f23afc7db004964bfc9f487ca3f6716d..cb800b22b87d384ed6eb3bcdc7a35f4e6314b39b 100644 (file)
@@ -29,7 +29,7 @@ class ChartPane extends ComponentBase {
         if (result.error)
             return;
 
-        var formatter = result.metric.makeFormatter(3);
+        var formatter = result.metric.makeFormatter(4);
         var self = this;
 
         var overviewOptions = ChartsPage.overviewChartOptions(formatter);
index 98bb3b1b47cd9338390de4fb5a14d57f737d727f..f27d9ec68a37795b4f41d4ade481f8bded7c4642 100644 (file)
@@ -108,6 +108,7 @@ class PageWithCharts extends PageWithHeading {
     static mainChartOptions(valueFormatter)
     {
         var options = this.dashboardOptions(valueFormatter);
+        options.axis.yAxisWidth = 5;
         options.selection = {
             lineStyle: '#f93',
             lineWidth: 2,