Dashboard should defaults to first dashboard page when summary page is not available.
authordewei_zhu@apple.com <dewei_zhu@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Aug 2019 18:02:45 +0000 (18:02 +0000)
committerdewei_zhu@apple.com <dewei_zhu@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Aug 2019 18:02:45 +0000 (18:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=200180

Reviewed by Ryosuke Niwa.

Fix a bug that charts page will become the default when there is no summary page but there
is at least one dashboard page.

* public/v3/main.js: Charts page and analysis category page should not be considered as summay pages.
(main):

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

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/v3/main.js

index 4b40e04..408a174 100644 (file)
@@ -1,3 +1,16 @@
+2019-07-26  Dewei Zhu  <dewei_zhu@apple.com>
+
+        Dashboard should defaults to first dashboard page when summary page is not available.
+        https://bugs.webkit.org/show_bug.cgi?id=200180
+
+        Reviewed by Ryosuke Niwa.
+
+        Fix a bug that charts page will become the default when there is no summary page but there
+        is at least one dashboard page.
+
+        * public/v3/main.js: Charts page and analysis category page should not be considered as summay pages.
+        (main):
+
 2019-07-30  Dewei Zhu  <dewei_zhu@apple.com>
 
         Analysis task page should show build request author and creation time.
index 0ebc2f0..57a06c3 100644 (file)
@@ -38,8 +38,6 @@ function main() {
         const chartsPage = new ChartsPage(chartsToolbar);
         const analysisCategoryPage = new AnalysisCategoryPage();
 
-        summaryPages.push(chartsPage);
-        summaryPages.push(analysisCategoryPage);
         if (testFreshnessPage)
             summaryPages.push(testFreshnessPage);
 
@@ -53,7 +51,7 @@ function main() {
         buildRequestQueuePage.setParentPage(analysisCategoryPage);
 
         const heading = new Heading(manifest.siteTitle);
-        heading.addPageGroup(summaryPages);
+        heading.addPageGroup([chartsPage, analysisCategoryPage, ...summaryPages]);
 
         heading.setTitle(manifest.siteTitle);
         heading.addPageGroup(dashboardPages);
@@ -61,6 +59,8 @@ function main() {
         const router = new PageRouter();
         for (const summaryPage of summaryPages)
             router.addPage(summaryPage);
+        router.addPage(chartsPage);
+        router.addPage(analysisCategoryPage);
         router.addPage(createAnalysisTaskPage);
         router.addPage(analysisTaskPage);
         router.addPage(buildRequestQueuePage);