Alerts fix (#177)

This PR fixes an issue with alerts where rails usage and defaults conflicted with our naming conventions.
RoleModel · Jun 7, 2023 · 5b58ada · 5b58ada
1 parent 05f0585
commit 5b58ada
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/components/alert.scss b/src/components/alert.scss
@@ -71,7 +71,9 @@
   }
 }
 
-.alert-danger {
+// alert-alert is an alias for alert-danger. See Alert.mdx for reasoning.
+.alert-danger,
+.alert-alert {
   @extend %alert-global;
 
   background-color: var(--op-color-alerts-danger-plus-eight);

diff --git a/src/stories/Components/Alert/Alert.mdx b/src/stories/Components/Alert/Alert.mdx
@@ -40,6 +40,8 @@ Alert can be used as a standalone component, however, it does have a few depende
 
 `.alert-danger` Provides a danger styled message
 
+`.alert-alert` is an alias for `.alert-danger`. Since RoleModel commonly uses [Rails](https://rubyonrails.org/) to develop projects, adding this alias allows this to be used for flash messages without having to change the default in rails. [Devise](https://github.com/heartcombo/devise) is also commonly used and cannot easily be configured away from this.
+
 <Canvas of={AlertStories.Danger} />
 
 ### Info