Fixes #5109 - Remove sync icon active and hover state (#5174)

mozilla-services · Nov 14, 2018 · 856958c · 856958c
1 parent 2a0bd54
commit 856958c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/server/src/pages/shotindex/view.js b/server/src/pages/shotindex/view.js
@@ -334,13 +334,13 @@ class Card extends React.Component {
     let favoriteIndicator = null;
     if (!shot.expireTime) {
       favoriteIndicator = <Localized id="shotIndexFavoriteIcon" attrs={{title: true}}>
-          <div className={classnames("indicator fav-shot", {"inactive": !this.props.hasFxa})}
+          <div className={classnames("indicator link fav-shot", {"inactive": !this.props.hasFxa})}
             onClick={this.props.hasFxa ? this.onClickFavorite.bind(this, shot) : null}
             title=""></div>
         </Localized>;
     } else if (this.props.hasFxa) {
       favoriteIndicator = <Localized id="shotIndexNonFavoriteIcon" attrs={{title: true}}>
-          <div className="indicator non-fav-shot"
+          <div className="indicator link non-fav-shot"
             onClick={this.onClickFavorite.bind(this, shot)} title=""></div>
         </Localized>;
     }

diff --git a/static/css/shot-index.scss b/static/css/shot-index.scss
@@ -222,11 +222,13 @@ $shot-width: 210px;
     background-color: $white;
     background-position: center;
     background-repeat: no-repeat;
-    &:hover {
-      background-color: $light-hover;
-    }
-    &:active {
-      background-color: $light-active;
+    &.link {
+      &:hover {
+        background-color: $light-hover;
+      }
+      &:active {
+        background-color: $light-active;
+      }
     }
   }