Merge branch 'new-sidebar-context-header-hover-color' into 'master'

Fixed sidebar context header hover colors

Closes #35999

See merge request !13323

1 files changed, 13 insertions, 24 deletions

diff --git a/app/assets/stylesheets/new_sidebar.scss b/app/assets/stylesheets/new_sidebar.scss
index 3d202183c82..6fd413dd7e3 100644
--- a/app/assets/stylesheets/new_sidebar.scss
+++ b/app/assets/stylesheets/new_sidebar.scss
@@ -8,8 +8,8 @@ $active-color: $indigo-700;
 $active-hover-background: $active-background;
 $active-hover-color: $gl-text-color;
 $inactive-badge-background: rgba(0, 0, 0, .08);
-$hover-background: $indigo-700;
-$hover-color: $white-light;
+$hover-background: $white-light;
+$hover-color: $gl-text-color;
 $inactive-color: $gl-text-color-secondary;
 $new-sidebar-width: 220px;
 
@@ -31,6 +31,7 @@ $new-sidebar-width: 220px;
 
 .context-header {
   position: relative;
+  margin-right: 2px;
 
   a {
     border-bottom: 1px solid $border-color;
@@ -43,22 +44,16 @@ $new-sidebar-width: 220px;
     @media (max-width: $screen-xs-max) {
       padding-right: 30px;
     }
+  }
 
-    &:hover {
-      background-color: $hover-background;
-      color: $hover-color;
-      border-color: $hover-background;
-
-      .avatar-container {
-        border-color: transparent;
-      }
-
-      .settings-avatar {
-        background-color: $indigo-500;
+  &:hover,
+  a:hover {
+    background-color: $hover-background;
+    color: $hover-color;
 
-        i {
-          color: $hover-color;
-        }
+    .settings-avatar {
+      i {
+        color: $hover-color;
       }
     }
   }
@@ -74,13 +69,6 @@ $new-sidebar-width: 220px;
     text-overflow: ellipsis;
   }
 
-
-  &:hover {
-    .close-nav-button {
-      color: $white-light;
-    }
-  }
-
   .close-nav-button {
     display: none;
     position: absolute;
@@ -90,6 +78,7 @@ $new-sidebar-width: 220px;
     background-color: transparent;
     border: 0;
     padding: 0 10px;
+    color: $gl-text-color-secondary;
 
     @media (max-width: $screen-xs-max) {
       display: block;
@@ -233,7 +222,7 @@ $new-sidebar-width: 220px;
     &:not(.active) {
       > a {
         margin-left: 1px;
-        margin-right: 3px;
+        margin-right: 2px;
       }
 
       .sidebar-sub-level-items {