fix(theme): fix dark theme bugs (patternfly#5535)

mattnolting · Dec 12, 2023 · 0339eba · 0339eba
1 parent 05eed77
commit 0339eba
Show file tree

Hide file tree

Showing 42 changed files with 444 additions and 339 deletions.
diff --git a/src/patternfly/assets/images/pf_logo.svg b/src/patternfly/assets/images/pf_logo.svg
diff --git a/src/patternfly/base/themes/dark/_variables.scss b/src/patternfly/base/themes/dark/_variables.scss
@@ -33,6 +33,8 @@
   --#{$pf-global}--active-color--100: #{$pf-v5-global--active-color--100};
   --#{$pf-global}--primary-color--100: #{$pf-v5-global--primary-color--100};
   --#{$pf-global}--primary-color--300: #{$pf-v5-global--primary-color--300};
+  --#{$pf-global}--primary-color--400: #{$pf-v5-global--primary-color--400};
+  --#{$pf-global}--custom-color--200: #{$pf-v5-global--custom-color--200};
   --#{$pf-global}--success-color--100: #{$pf-v5-global--success-color--100};
   --#{$pf-global}--warning-color--100: #{$pf-v5-global--warning-color--100};
   --#{$pf-global}--warning-color--200: #{$pf-v5-global--warning-color--200};

diff --git a/src/patternfly/components/Alert/themes/dark/alert.scss b/src/patternfly/components/Alert/themes/dark/alert.scss
@@ -2,16 +2,16 @@
 
 @mixin pf-v5-theme-dark-alert() {
   .#{$alert} {
-    --#{$alert}--BackgroundColor: var(--#{$pf-global}--palette--black-600);
-    --#{$alert}__title--Color: var(--#{$pf-global}--custom-color--200);
+    --#{$alert}--BackgroundColor: var(--#{$pf-global}--BackgroundColor--300);
+    --#{$alert}--m-custom__title--Color: var(--#{$pf-global}--custom-color--200);
     --#{$alert}--m-success__title--Color: var(--#{$pf-global}--success-color--100);
     --#{$alert}--m-danger__title--Color: var(--#{$pf-global}--danger-color--100);
     --#{$alert}--m-warning__title--Color: var(--#{$pf-global}--warning-color--100);
     --#{$alert}--m-info__title--Color: var(--#{$pf-global}--info-color--100);
-    --#{$alert}--m-inline--BackgroundColor: var(--#{$pf-global}--palette--black-600);
-    --#{$alert}--m-inline--m-success--BackgroundColor: var(--#{$pf-global}--palette--black-600);
-    --#{$alert}--m-inline--m-danger--BackgroundColor: var(--#{$pf-global}--palette--black-600);
-    --#{$alert}--m-inline--m-warning--BackgroundColor: var(--#{$pf-global}--palette--black-600);
-    --#{$alert}--m-inline--m-info--BackgroundColor: var(--#{$pf-global}--palette--black-600);
+    --#{$alert}--m-inline--m-custom--BackgroundColor: var(--#{$pf-global}--BackgroundColor--300);
+    --#{$alert}--m-inline--m-success--BackgroundColor: var(--#{$pf-global}--BackgroundColor--300);
+    --#{$alert}--m-inline--m-danger--BackgroundColor: var(--#{$pf-global}--BackgroundColor--300);
+    --#{$alert}--m-inline--m-warning--BackgroundColor: var(--#{$pf-global}--BackgroundColor--300);
+    --#{$alert}--m-inline--m-info--BackgroundColor: var(--#{$pf-global}--BackgroundColor--300);
   }
 }
diff --git a/src/patternfly/components/AppLauncher/app-launcher.scss b/src/patternfly/components/AppLauncher/app-launcher.scss
@@ -186,6 +186,7 @@
   font-weight: var(--#{$app-launcher}__menu-item--FontWeight);
   color: var(--#{$app-launcher}__menu-item--Color);
   white-space: nowrap;
+  background: none;
   border: 0;
 
   &:hover,

diff --git a/src/patternfly/components/Badge/themes/dark/badge.scss b/src/patternfly/components/Badge/themes/dark/badge.scss
@@ -3,6 +3,7 @@
 @mixin pf-v5-theme-dark-badge() {
   .#{$badge} {
     --#{$badge}--m-read--BackgroundColor: var(--#{$pf-global}--palette--black-500);
+    --#{$badge}--m-unread--Color: var(--#{$pf-global}--primary-color--400);
     --#{$badge}--m-unread--BackgroundColor: var(--#{$pf-global}--primary-color--300);
   }
 }
diff --git a/src/patternfly/components/Banner/banner.scss b/src/patternfly/components/Banner/banner.scss
@@ -26,9 +26,9 @@
 
   // modifier variables
   --#{$banner}--m-blue--BackgroundColor: var(--#{$pf-global}--palette--blue-200);
-  --#{$banner}--m-red--BackgroundColor: var(--#{$pf-global}--palette--red-100);
-  --#{$banner}--m-green--BackgroundColor: var(--#{$pf-global}--palette--green-500);
-  --#{$banner}--m-gold--BackgroundColor: var(--#{$pf-global}--palette--gold-400);
+  --#{$banner}--m-red--BackgroundColor: var(--#{$pf-global}--danger-color--100);
+  --#{$banner}--m-green--BackgroundColor: var(--#{$pf-global}--success-color--100);
+  --#{$banner}--m-gold--BackgroundColor: var(--#{$pf-global}--warning-color--100);
   --#{$banner}--m-sticky--ZIndex: var(--#{$pf-global}--ZIndex--md);
   --#{$banner}--m-sticky--BoxShadow: var(--#{$pf-global}--BoxShadow--md-bottom);
 
@@ -43,7 +43,7 @@
   background-color: var(--#{$banner}--BackgroundColor);
 
   &.pf-m-blue {
-    @include pf-v5-t-light;
+    @include pf-v5-t-light(false);
 
     --#{$banner}--BackgroundColor: var(--#{$banner}--m-blue--BackgroundColor);
   }
@@ -57,7 +57,7 @@
   }
 
   &.pf-m-gold {
-    @include pf-v5-t-light;
+    @include pf-v5-t-light(false);
 
     --#{$banner}--BackgroundColor: var(--#{$banner}--m-gold--BackgroundColor);
   }

diff --git a/src/patternfly/components/Banner/themes/dark/banner.scss b/src/patternfly/components/Banner/themes/dark/banner.scss
@@ -2,17 +2,13 @@
 
 @mixin pf-v5-theme-dark-banner() {
   .#{$banner} {
-    &.pf-m-info,
     &.pf-m-blue,
-    &.pf-m-danger,
     &.pf-m-red,
-    &.pf-m-success,
     &.pf-m-green,
-    &.pf-m-warning,
     &.pf-m-gold {
-      color: var(--#{$pf-global}--palette--black-900);
+      --#{$banner}--Color: var(--#{$pf-global}--palette--black-900);
     }
 
-    @include pf-v5-theme-dark--t-dark;
+    @include pf-v5-theme-dark--t-dark(false);
   }
 }
diff --git a/src/patternfly/components/Button/button.scss b/src/patternfly/components/Button/button.scss
@@ -111,6 +111,7 @@
   --#{$button}--m-link--active--BackgroundColor: transparent;
   --#{$button}--m-link--active--Color: var(--#{$pf-global}--link--Color--hover);
   --#{$button}--m-link--disabled--BackgroundColor: transparent;
+  --#{$button}--m-link--disabled--Color: var(--#{$pf-global}--disabled-color--100);
   --#{$button}--m-link--m-inline--FontSize: inherit;
   --#{$button}--m-link--m-inline--hover--TextDecoration: var(--#{$pf-global}--link--TextDecoration--hover);
   --#{$button}--m-link--m-inline--hover--Color: var(--#{$pf-global}--link--Color--hover);
@@ -352,6 +353,7 @@
   // Link buttons
   &.pf-m-link {
     --#{$button}--disabled--BackgroundColor: var(--#{$button}--m-link--disabled--BackgroundColor);
+    --#{$button}--disabled--Color: var(--#{$button}--m-link--disabled--Color);
 
     color: var(--#{$button}--m-link--Color);
     background-color: var(--#{$button}--m-link--BackgroundColor);

diff --git a/src/patternfly/components/Button/themes/dark/button.scss b/src/patternfly/components/Button/themes/dark/button.scss
@@ -3,9 +3,10 @@
 @mixin pf-v5-theme-dark-button() {
   .#{$button} {
     --#{$button}--disabled--Color: var(--#{$pf-global}--disabled-color--300);
-    --#{$button}--m-primary--BackgroundColor: var(--#{$pf-global}--primary-color--300); // should blue-400 be a primary-color?
+    --#{$button}--m-primary--BackgroundColor: var(--#{$pf-global}--primary-color--300);
+    --#{$button}--m-primary--Color: var(--#{$pf-global}--primary-color--400);
     --#{$button}--m-tertiary--after--BorderColor: var(--#{$pf-global}--BorderColor--100);
-    --#{$button}--m-tertiary--Color: var(--#{$pf-global}--palette--black-100); // global variable for this?
+    --#{$button}--m-tertiary--Color: var(--#{$pf-global}--palette--black-100);
     --#{$button}--m-tertiary--hover--after--BorderColor: var(--#{$pf-global}--BorderColor--100);
     --#{$button}--m-tertiary--hover--Color: var(--#{$pf-global}--palette--black-100);
     --#{$button}--m-tertiary--focus--after--BorderColor: var(--#{$pf-global}--BorderColor--100);
@@ -23,6 +24,7 @@
     --#{$button}--m-danger--hover--Color: var(--#{$pf-global}--palette--black-900);
     --#{$button}--m-danger--focus--Color: var(--#{$pf-global}--palette--black-900);
     --#{$button}--m-danger--active--Color: var(--#{$pf-global}--palette--black-900);
+    --#{$button}--m-link--disabled--Color: var(--#{$pf-global}--disabled-color--100);
     --#{$button}--m-control--BackgroundColor: var(--#{$pf-global}--BackgroundColor--400);
     --#{$button}--m-control--hover--BackgroundColor: var(--#{$pf-global}--BackgroundColor--400);
     --#{$button}--m-control--active--BackgroundColor: var(--#{$pf-global}--BackgroundColor--400);

diff --git a/src/patternfly/components/CalendarMonth/calendar-month.scss b/src/patternfly/components/CalendarMonth/calendar-month.scss
@@ -1,7 +1,4 @@
-// @debug $calendar-month; // check your variable names located in src/patternfly/sass-utilities/component-namespaces.scss
-
 .#{$calendar-month} {
-  --#{$calendar-month}--BackgroundColor: var(--#{$pf-global}--BackgroundColor--100);
   --#{$calendar-month}--PaddingTop: var(--#{$pf-global}--spacer--lg);
   --#{$calendar-month}--PaddingRight: var(--#{$pf-global}--spacer--lg);
   --#{$calendar-month}--PaddingBottom: var(--#{$pf-global}--spacer--md);
@@ -49,7 +46,7 @@
   --#{$calendar-month}__dates-cell--m-in-range--m-end-range--before--Right: 50%;
   --#{$calendar-month}__dates-cell--m-in-range__date--hover--BackgroundColor: var(--#{$pf-global}--palette--blue-100);
   --#{$calendar-month}__dates-cell--m-in-range__date--focus--BackgroundColor: var(--#{$pf-global}--palette--blue-100);
-  --#{$calendar-month}__dates-cell--m-adjacent-month__date--Color: var(--#{$pf-global}--disabled-color--100);
+  --#{$calendar-month}__dates-cell--m-adjacent-month__date--Color: var(--#{$pf-global}--Color--200);
 
   // date button
   --#{$calendar-month}__date--Width: 4ch;
@@ -71,7 +68,6 @@
   flex-direction: column;
   padding: var(--#{$calendar-month}--PaddingTop) var(--#{$calendar-month}--PaddingRight) var(--#{$calendar-month}--PaddingBottom) var(--#{$calendar-month}--PaddingLeft);
   font-size: var(--#{$calendar-month}--FontSize);
-  background-color: var(--#{$calendar-month}--BackgroundColor);
 }
 
 .#{$calendar-month}__header {

diff --git a/src/patternfly/components/CalendarMonth/themes/dark/calendar-month.scss b/src/patternfly/components/CalendarMonth/themes/dark/calendar-month.scss
@@ -4,6 +4,7 @@
   .#{$calendar-month} {
     --#{$calendar-month}__dates-cell--m-current__date--BackgroundColor: var(--#{$pf-global}--palette--black-500);
     --#{$calendar-month}__dates-cell--m-selected__date--BackgroundColor: var(--#{$pf-global}--primary-color--300);
+    --#{$calendar-month}__dates-cell--m-selected__date--Color: var(--#{$pf-global}--primary-color--400);
     --#{$calendar-month}__dates-cell--m-selected__date--hover--BackgroundColor: var(--#{$pf-global}--palette--black-900);
     --#{$calendar-month}__dates-cell--m-selected__date--focus--BackgroundColor: var(--#{$pf-global}--palette--blue-400);
     --#{$calendar-month}__dates-cell--m-in-range--before--BackgroundColor: var(--#{$pf-global}--BackgroundColor--200);
@@ -26,6 +27,7 @@
       --#{$calendar-month}__date--after--BorderColor: var(--#{$pf-global}--active-color--100);
       --#{$calendar-month}__date--after--BorderWidth: var(--#{$pf-global}--BorderWidth--sm);
       --#{$calendar-month}__date--BackgroundColor: var(--#{$pf-global}--primary-color--300);
+      --#{$calendar-month}__date--Color: var(--#{$pf-global}--primary-color--400);
     }
   }
 }
diff --git a/src/patternfly/components/DragDrop/drag-drop.scss b/src/patternfly/components/DragDrop/drag-drop.scss
@@ -36,34 +36,53 @@
 }
 
 .#{$droppable} {
-  $pf-v5-global--BackgroundColor--100: #fff;
-  $pf-v5-c-droppable--m-dragging--after--BackgroundColor: rgba($pf-v5-global--BackgroundColor--100, .6);
-
-  --#{$droppable}--m-dragging--after--BackgroundColor: #{$pf-v5-c-droppable--m-dragging--after--BackgroundColor}; // allows for rbga() so we can combine background and border in single element
+  --#{$droppable}--before--BackgroundColor: transparent;
+  --#{$droppable}--before--Opacity: 0;
+  --#{$droppable}--after--BorderWidth: 0;
+  --#{$droppable}--after--BorderColor: transparent;
+  --#{$droppable}--m-dragging--before--BackgroundColor: var(--#{$pf-global}--palette--white);
+  --#{$droppable}--m-dragging--before--Opacity: .6;
   --#{$droppable}--m-dragging--after--BorderWidth: var(--#{$pf-global}--BorderWidth--sm);
   --#{$droppable}--m-dragging--after--BorderColor: var(--#{$pf-global}--active-color--100);
   --#{$droppable}--m-drag-outside--after--BorderColor: var(--#{$pf-global}--danger-color--100);
 
-  &.pf-m-dragging {
-    position: relative;
+  &::before,
+  &::after {
+    position: absolute;
+    top: 0;
+    right: 0;
+    bottom: 0;
+    left: 0;
+    pointer-events: none;
+    content: "";
+  }
 
-    &::after {
-      position: absolute;
-      top: 0;
-      right: 0;
-      bottom: 0;
-      left: 0;
-      pointer-events: none;
-      content: "";
-    }
+  &::before {
+    background-color: var(--#{$droppable}--before--BackgroundColor);
+    opacity: var(--#{$droppable}--before--Opacity);
   }
 
   &::after {
-    background-color: var(--#{$droppable}--m-dragging--after--BackgroundColor);
-    border: var(--#{$droppable}--m-dragging--after--BorderWidth) solid var(--#{$droppable}--m-dragging--after--BorderColor);
+    border: var(--#{$droppable}--after--BorderWidth, 0) solid var(--#{$droppable}--after--BorderColor, transparent);
+  }
+
+  &.pf-m-dragging {
+    --#{$droppable}--before--BackgroundColor: var(--#{$droppable}--m-dragging--before--BackgroundColor);
+    --#{$droppable}--before--Opacity: var(--#{$droppable}--m-dragging--before--Opacity);
+    --#{$droppable}--after--BorderWidth: var(--#{$droppable}--m-dragging--after--BorderWidth);
+    --#{$droppable}--after--BorderColor: var(--#{$droppable}--m-dragging--after--BorderColor);
+
+    position: relative;
   }
 
   &.pf-m-drag-outside {
     --#{$droppable}--m-dragging--after--BorderColor: var(--#{$droppable}--m-drag-outside--after--BorderColor);
   }
 }
+
+// stylelint-disable no-invalid-position-at-import-rule
+@import "themes/dark/drag-drop";
+
+@include pf-v5-theme-dark {
+  @include pf-v5-theme-dark-drag-drop;
+}
diff --git a/src/patternfly/components/DragDrop/themes/dark/drag-drop.scss b/src/patternfly/components/DragDrop/themes/dark/drag-drop.scss
@@ -0,0 +1,7 @@
+@import "../../../../sass-utilities/themes/dark/all";
+
+@mixin pf-v5-theme-dark-drag-drop() {
+  .#{$droppable} {
+    --#{$droppable}--m-dragging--before--Opacity: .2;
+  }
+}
diff --git a/src/patternfly/components/Dropdown/themes/dark/dropdown.scss b/src/patternfly/components/Dropdown/themes/dark/dropdown.scss
@@ -8,7 +8,9 @@
     --#{$dropdown}__toggle--before--BorderBottomColor: var(--#{$pf-global}--BorderColor--400);
     --#{$dropdown}__toggle--before--BorderLeftColor: transparent;
     --#{$dropdown}__toggle--disabled--BackgroundColor: var(--#{$pf-global}--palette--black-500);
+    --#{$dropdown}__toggle--m-split-button--m-primary--child--BackgroundColor: var(--#{$pf-global}--primary-color--300);
     --#{$dropdown}__toggle--m-primary--BackgroundColor: var(--#{$pf-global}--primary-color--300);
+    --#{$dropdown}__toggle--m-primary--Color: var(--#{$pf-global}--primary-color--400);
     --#{$dropdown}__menu--BackgroundColor: var(--#{$pf-global}--BackgroundColor--300);
     --#{$dropdown}__menu--Top: 100%;
     --#{$dropdown}--m-top__menu--TranslateY: -100%;

diff --git a/src/patternfly/components/DualListSelector/dual-list-selector.scss b/src/patternfly/components/DualListSelector/dual-list-selector.scss
@@ -34,9 +34,9 @@ $pf-v5-c-dual-list-selector__item--MaxNesting: 10;
   // List item
   --#{$dual-list-selector}__list-item-row--FontSize: var(--#{$pf-global}--FontSize--sm);
   --#{$dual-list-selector}__list-item-row--BackgroundColor: transparent;
-  --#{$dual-list-selector}__list-item-row--hover--BackgroundColor: var(--#{$pf-global}--BackgroundColor--light-300);
-  --#{$dual-list-selector}__list-item-row--focus-within--BackgroundColor: var(--#{$pf-global}--BackgroundColor--light-300);
-  --#{$dual-list-selector}__list-item-row--m-selected--BackgroundColor: var(--#{$pf-global}--BackgroundColor--light-300);
+  --#{$dual-list-selector}__list-item-row--hover--BackgroundColor: var(--#{$pf-global}--BackgroundColor--200);
+  --#{$dual-list-selector}__list-item-row--focus-within--BackgroundColor: var(--#{$pf-global}--BackgroundColor--200);
+  --#{$dual-list-selector}__list-item-row--m-selected--BackgroundColor: var(--#{$pf-global}--BackgroundColor--200);
   --#{$dual-list-selector}__list-item--m-ghost-row--BackgroundColor: var(--#{$pf-global}--BackgroundColor--100);
   --#{$dual-list-selector}__list-item--m-ghost-row--Opacity: .4;
 
@@ -332,3 +332,10 @@ $pf-v5-c-dual-list-selector__item--MaxNesting: 10;
   }
 }
 // stylelint-enable
+
+// stylelint-disable no-invalid-position-at-import-rule
+@import "themes/dark/dual-list-selector";
+
+@include pf-v5-theme-dark {
+  @include pf-v5-theme-dark-dual-list-selector;
+}
diff --git a/src/patternfly/components/DualListSelector/examples/DualListSelector.md b/src/patternfly/components/DualListSelector/examples/DualListSelector.md
@@ -251,7 +251,7 @@ cssPrefix: pf-v5-c-dual-list-selector
     {{/dual-list-selector-tools}}
     {{#> dual-list-selector-status}}
       {{#> dual-list-selector-status-text}}
-        1 of 5 items selected
+        3 of 5 items selected
       {{/dual-list-selector-status-text}}
     {{/dual-list-selector-status}}
     {{#> dual-list-selector-menu}}

diff --git a/src/patternfly/components/DualListSelector/themes/dark/dual-list-selector.scss b/src/patternfly/components/DualListSelector/themes/dark/dual-list-selector.scss
@@ -0,0 +1,9 @@
+@import "../../../../sass-utilities/themes/dark/all";
+
+@mixin pf-v5-theme-dark-dual-list-selector() {
+  .#{$dual-list-selector} {
+    --#{$dual-list-selector}__list-item-row--hover--BackgroundColor: var(--#{$pf-global}--BackgroundColor--400);
+    --#{$dual-list-selector}__list-item-row--focus-within--BackgroundColor: var(--#{$pf-global}--BackgroundColor--400);
+    --#{$dual-list-selector}__list-item-row--m-selected--BackgroundColor: var(--#{$pf-global}--BackgroundColor--400);
+  }
+}