Merge pull request #3455 from danny007in/sidebar-light-color-fix

fix sidebar light text color

.bundlewatch.config.json

@@ -22,7 +22,7 @@
     },
     {
       "path": "./dist/css/alt/adminlte.core.min.css",
-      "maxSize": "50.8 kB"
+      "maxSize": "51.1 kB"
     },
     {
       "path": "./dist/css/alt/adminlte.extra-components.css",

build/scss/_main-sidebar.scss

@@ -302,6 +302,12 @@
     > .nav-item {
       > .nav-link {
         color: $sidebar-light-submenu-color;
+
+        &:hover,
+        &:focus {
+          background-color: $sidebar-light-submenu-hover-bg;
+          color: $sidebar-light-submenu-hover-color;
+        }
       }
 
       > .nav-link.active {

build/scss/_navs.scss

@@ -98,11 +98,12 @@
   @include navbar-variant($color, $value);
 }
 
-.dark-mode {
-  .nav-pills .nav-link {
+.dark-mode:not([class*="sidebar-light-"]) {
+  &.nav-pills .nav-link {
     color: $gray-400;
   }
-  .nav-tabs {
+
+  &.nav-tabs {
     border-color: lighten($dark, 15%);
 
     .nav-link:focus,

build/scss/_variables.scss

@@ -129,6 +129,7 @@ $sidebar-light-hover-color: $gray-900 !default;
 $sidebar-light-active-color: $black !default;
 $sidebar-light-submenu-bg: transparent !default;
 $sidebar-light-submenu-color: #777 !default;
+$sidebar-light-submenu-hover-color: $black !default;
 $sidebar-light-submenu-hover-bg: $sidebar-light-hover-bg !default;
 $sidebar-light-submenu-active-color: $sidebar-light-hover-color !default;
 $sidebar-light-submenu-active-bg: $sidebar-light-submenu-hover-bg !default;