results.webkit.org: Fix drawer style
authorjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Aug 2019 05:33:33 +0000 (05:33 +0000)
committerjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Aug 2019 05:33:33 +0000 (05:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=200988

Rubber-stamped by Aakash Jain.

* resultsdbpy/resultsdbpy/view/static/js/drawer.js:
(Drawer): Nothing in the drawer should be selectable.
(ConfigurationSelectors): Standardize text size, remove underscore from expanders.
* resultsdbpy/resultsdbpy/view/static/library/css/webkit.css:
(.action:hover): Disable underline on 'links' which are actually actions
(.unselectable): Make a div unselectable.
* resultsdbpy/resultsdbpy/view/static/library/js/components/TimelineComponents.js:
(Timeline.ExpandableSeriesWithHeaderExpanderComponent): Remove underscore from expanders.

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

Tools/ChangeLog
Tools/resultsdbpy/resultsdbpy/view/static/js/drawer.js
Tools/resultsdbpy/resultsdbpy/view/static/library/css/webkit.css
Tools/resultsdbpy/resultsdbpy/view/static/library/js/components/TimelineComponents.js

index 169734e..6e1dbd2 100644 (file)
@@ -1,3 +1,19 @@
+2019-08-21  Jonathan Bedard  <jbedard@apple.com>
+
+        results.webkit.org: Fix drawer style
+        https://bugs.webkit.org/show_bug.cgi?id=200988
+
+        Rubber-stamped by Aakash Jain.
+
+        * resultsdbpy/resultsdbpy/view/static/js/drawer.js:
+        (Drawer): Nothing in the drawer should be selectable.
+        (ConfigurationSelectors): Standardize text size, remove underscore from expanders.
+        * resultsdbpy/resultsdbpy/view/static/library/css/webkit.css:
+        (.action:hover): Disable underline on 'links' which are actually actions
+        (.unselectable): Make a div unselectable.
+        * resultsdbpy/resultsdbpy/view/static/library/js/components/TimelineComponents.js:
+        (Timeline.ExpandableSeriesWithHeaderExpanderComponent): Remove underscore from expanders.
+
 2019-08-21  Fujii Hironori  <Hironori.Fujii@sony.com>
 
         [Win][MiniBrowser] Add a progress indicator to the main window
index 4b20ba4..2388b07 100644 (file)
@@ -59,7 +59,7 @@ function Drawer(controls = []) {
         }
     });
 
-    return `<div class="drawer left under-topbar-with-actions" ref="${drawerRef}">
+    return `<div class="drawer left under-topbar-with-actions unselectable" ref="${drawerRef}">
             ${controls.map(control => {
                 return `<div class="list">
                         <div class="item">${control}</div>
@@ -226,7 +226,7 @@ function ConfigurationSelectors(callback) {
                     }
                 });
 
-                DOM.inject(element, `<a style="cursor: pointer;" class="text medium" ref="${expander}">+</a>
+                DOM.inject(element, `<a class="link-button text medium" ref="${expander}">+</a>
                     ${details.name} <br>
                     ${options.map(option => {
                         let isChecked = false;
@@ -275,7 +275,7 @@ function ConfigurationSelectors(callback) {
             ref.setState(configurations);
         });
 
-        return `<div ref="${ref}"></div>`;
+        return `<div style="font-size: var(--smallSize);" ref="${ref}"></div>`;
     }).join('')
 }
 
index 7b964e1..f6b5d56 100644 (file)
@@ -194,6 +194,11 @@ a:hover {
   text-decoration: underline;
 }
 
+.link-button:hover {
+  text-decoration: none;
+  cursor: pointer;
+}
+
 .divider, hr {
   margin-block-start: 0.5em;
   margin-block-end: 0.5em;
@@ -2555,3 +2560,9 @@ pre {
 .progress-bar-container .progress-bar.error {
   background: var(--orange);
 }
+
+.unselectable {
+  -webkit-touch-callout: none;
+  -webkit-user-select: none;
+  user-select: none;
+}
index eeb1ee8..964f6a0 100644 (file)
@@ -535,7 +535,7 @@ Timeline.ExpandableSeriesWithHeaderExpanderComponent = (mainSeriesWithLable, ...
         })
     }));
     return {
-        header: Timeline.ExpandableHeaderComponent(`<a href="javascript:void(0)" ref="${ref}">+</a>` + mainLabel, subLabels, composer),
+        header: Timeline.ExpandableHeaderComponent(`<a class="link-button" href="javascript:void(0)" ref="${ref}">+</a>` + mainLabel, subLabels, composer),
         series: Timeline.ExpandableSeriesComponent(mainSeries, subSerieses, composer),
     }
 }