fix: reverting app-level orange alerts to intended color (#3612)

• During the SASS variable cleanup, I made updates to the orange color palette that broke the a11y contrast of app-level alerts • This change fixes that regression Tested in: ✔︎ Chrome ✔︎ Firefox Fixes #3604 Signed-off-by: Scott Mathis <[email protected]>
clovercloudsolutions · Jul 11, 2019 · a4b63da · a4b63da
1 parent 61155a2
commit a4b63da
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/gemini/screens/alert/app-level-static/default/chrome.png b/gemini/screens/alert/app-level-static/default/chrome.png
diff --git a/src/clr-angular/emphasis/alert/_variables.alert.scss b/src/clr-angular/emphasis/alert/_variables.alert.scss
@@ -105,7 +105,8 @@ $clr-app-alert-info-close-icon-hover-opacity: $clr-app-alert-close-icon-hover-op
 
 // App Warning type
 // Usage: ./_alert.clarity.scss
-$clr-app-alert-warning-bg-color: $clr-color-warning-800 !default;
+// TODO-UX: why is the next color not in our color palettes?
+$clr-app-alert-warning-bg-color: #c25400 !default;
 $clr-app-alert-warning-icon-color: $clr-app-alert-close-icon-color !default;
 $clr-app-alert-warning-border-color: none !default;
 $clr-app-alert-warning-font-color: $clr-app-level-alert-color !default;