fix(topsites): Fixes #1154 TopSites tile loses highlight on hover

mozilla · Jan 27, 2017 · d43cc0c · d43cc0c
1 parent 8683703
commit d43cc0c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/content-src/components/SiteIcon/SiteIcon.scss b/content-src/components/SiteIcon/SiteIcon.scss
@@ -49,8 +49,4 @@
     overflow: hidden;
     z-index: 1;
   }
-
-  &:hover .site-icon-title {
-    background-color: $tile-title-hover-color;
-  }
 }
diff --git a/content-src/components/TopSites/TopSites.scss b/content-src/components/TopSites/TopSites.scss
@@ -9,7 +9,9 @@
 
   // This is a container for the delete menu
   .tile-outer {
+    @include item-shadow;
     @include link-menu-button;
+    border-radius: $border-radius;
     position: relative;
     display: inline-block;
     margin: 0 $tile-gutter 0 0;
@@ -29,6 +31,10 @@
         box-shadow: none;
       }
     }
+
+    &:hover .site-icon-title {
+      background-color: $tile-title-hover-color;
+    }
   }
 
   .screenshot {
@@ -43,10 +49,8 @@
   }
 
   .tile {
-    @include item-shadow;
     display: inline-flex;
     flex-shrink: 0;
-    border-radius: $border-radius;
     flex-direction: column;
     font-size: $tile-font-size;
     height: $tile-height;
@@ -96,7 +100,6 @@
   .top-sites {
 
     .tile {
-      @include item-shadow;
       box-shadow: none;
 
       .tile-img-container {