New perf dashboard doesn't preserve the number of days when clicking on a dashboard...
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 5 Feb 2015 03:36:21 +0000 (03:36 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 5 Feb 2015 03:36:21 +0000 (03:36 +0000)
https://bugs.webkit.org/show_bug.cgi?id=141280

Reviewed by Chris Dumez.

Fixed the bug by passing in "since" as a query parameter to the charts page.

Also fixed the styling issue that manifests when a JSON fetching fails on "Dashboard" page.

* public/v2/app.css: Fixed CSS rules for error messages shown in the place of charts.
* public/v2/app.js:
(App.IndexController): Changed the default number of days from one month to one week.
(App.IndexController._sharedDomainChanged): Set "since" property on the controller.
* public/v2/index.html: Pass in "since" property on the controller as a query parameter.

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

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/v2/app.css
Websites/perf.webkit.org/public/v2/app.js
Websites/perf.webkit.org/public/v2/index.html

index 1238803..edec13c 100644 (file)
@@ -1,5 +1,22 @@
 2015-02-04  Ryosuke Niwa  <rniwa@webkit.org>
 
+        New perf dashboard doesn't preserve the number of days when clicking on a dashboard chart
+        https://bugs.webkit.org/show_bug.cgi?id=141280
+
+        Reviewed by Chris Dumez.
+
+        Fixed the bug by passing in "since" as a query parameter to the charts page.
+
+        Also fixed the styling issue that manifests when a JSON fetching fails on "Dashboard" page.
+
+        * public/v2/app.css: Fixed CSS rules for error messages shown in the place of charts.
+        * public/v2/app.js:
+        (App.IndexController): Changed the default number of days from one month to one week.
+        (App.IndexController._sharedDomainChanged): Set "since" property on the controller.
+        * public/v2/index.html: Pass in "since" property on the controller as a query parameter.
+
+2015-02-04  Ryosuke Niwa  <rniwa@webkit.org>
+
         New perf dashboard erroneously clears zoom when poping history items
         https://bugs.webkit.org/show_bug.cgi?id=141278
 
index 45fa890..51cb5c8 100755 (executable)
@@ -403,13 +403,19 @@ table.dashboard tbody td .chart {
 }
 
 table.dashboard tbody td .chart,
-table.dashboard tbody td .failure,
 table.dashboard tbody td .progress {
     display: inline-block;
     width: 100%;
     height: 12rem;
     line-height: 12rem;
-    vertical-align: center;
+    vertical-align: middle;
+}
+
+table.dashboard tbody td .failure {
+    display: table-cell;
+    width: 100%;
+    height: 12rem;
+    vertical-align: middle;
 }
 
 #analysis-tasks,
index 0ed6dac..6d6d803 100755 (executable)
@@ -59,7 +59,7 @@ App.IndexController = Ember.Controller.extend({
     _previousGrid: {},
     headerColumns: [],
     rows: [],
-    numberOfDays: 30,
+    numberOfDays: 7,
     editMode: false,
 
     gridChanged: function ()
@@ -119,6 +119,7 @@ App.IndexController = Ember.Controller.extend({
         numberOfDays = parseInt(numberOfDays);
         var present = Date.now();
         var past = present - numberOfDays * 24 * 3600 * 1000;
+        this.set('since', past);
         this.set('sharedDomain', [past, present]);
     }.observes('numberOfDays').on('init'),
 
index a092852..0147cf7 100755 (executable)
@@ -78,7 +78,7 @@
                                 {{/if}}
                             {{else}}
                                 {{#if chartData}}
-                                    {{#link-to 'charts' (query-params paneList=paneList)}}
+                                    {{#link-to 'charts' (query-params paneList=paneList since=controller.since)}}
                                     {{interactive-chart
                                         chartData=chartData
                                         domain=controller.sharedDomain