fix(server): smaller font size for upper and lower limits

GoogleChrome · Oct 24, 2019 · ead96e7 · ead96e7
1 parent 91a4c7a
commit ead96e7
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 4 deletions.
diff --git a/packages/server/src/ui/routes/build-view/audit-list/numeric-diff.css b/packages/server/src/ui/routes/build-view/audit-list/numeric-diff.css
@@ -5,6 +5,8 @@
  */
 
 .audit-numeric-diff {
+  --numeric-diff-bar-height: calc(var(--base-font-size) + 4px);
+
   display: flex;
   flex-direction: row;
   font-family: var(--monospace-font-family);
@@ -58,8 +60,8 @@
   bottom: 0;
   padding: 0 calc(var(--base-spacing) / 2);
 
-  font-size: calc(var(--base-font-size) * 3 / 4);
-  line-height: calc(var(--base-font-size) * 3 / 4);
+  font-size: var(--base-font-size);
+  line-height: var(--numeric-diff-bar-height);
   white-space: nowrap;
   font-weight: var(--medium-font-weight);
 
@@ -88,6 +90,8 @@
 .audit-numeric-diff__right-label {
   width: 60px;
   color: var(--secondary-text-color);
+  font-size: var(--subtext-font-size);
+  line-height: var(--numeric-diff-bar-height);
 }
 
 .audit-numeric-diff__left-label {

diff --git a/packages/server/src/ui/routes/build-view/build-hash-selector.css b/packages/server/src/ui/routes/build-view/build-hash-selector.css
@@ -43,6 +43,11 @@
   background: var(--compare-highlight-color);
 }
 
+.build-hash-selector--base li:hover {
+  cursor: pointer;
+  background: var(--neutral-highlight-color);
+}
+
 .build-hash-selector__list li.build-hash-selector__label-li:hover {
   cursor: initial;
   background: none;

diff --git a/packages/server/src/ui/routes/build-view/build-hash-selector.jsx b/packages/server/src/ui/routes/build-view/build-hash-selector.jsx
@@ -144,7 +144,7 @@ const BuildHashSelector_ = props => {
 
   return (
     <div className="container">
-      <ul className="build-hash-selector__list">
+      <ul className={`build-hash-selector__list build-hash-selector--${props.selector}`}>
         {builds.map((build, index) => (
           <Fragment key={build.id}>
             <BuildLineItem
@@ -153,7 +153,7 @@ const BuildHashSelector_ = props => {
               compareBuild={props.build}
               baseBuild={props.ancestorBuild}
               selector={props.selector}
-              withDevLine={index <= indexOfFirstDev && build.branch !== 'master'}
+              withDevLine={index <= indexOfFirstDev && props.build.branch !== 'master'}
               withDevBranchArc={index === indexOfFirstDev + 1}
             />
             {index === indexOfFirstDev && build.branch !== 'master' ? (