fix: error icon, add only relevant variants (#5014)

https://linear.app/unleash/issue/2-1508/fix-message-banner-error-variant-icon After aligning with @nicolaesocaciu, we fixed the `error` variant icon and removed irrelevant message variants for now. Co-authored-by: Nicolae <[email protected]>
Unleash · Oct 12, 2023 · 2adfb7b · 2adfb7b
1 parent 5f6c17f
commit 2adfb7b
Showing 1 changed file with 9 additions and 12 deletions.
diff --git a/frontend/src/component/messageBanners/MessageBanner/MessageBanner.tsx b/frontend/src/component/messageBanners/MessageBanner/MessageBanner.tsx
@@ -1,4 +1,9 @@
-import { Check, Close, InfoOutlined, WarningAmber } from '@mui/icons-material';
+import {
+    Check,
+    ErrorOutlineRounded,
+    InfoOutlined,
+    WarningAmber,
+} from '@mui/icons-material';
 import { styled, Icon, Link } from '@mui/material';
 import { usePlausibleTracker } from 'hooks/usePlausibleTracker';
 import { useNavigate } from 'react-router-dom';
@@ -37,13 +42,7 @@ const StyledIcon = styled('div', {
     color: theme.palette[variant].main,
 }));
 
-type BannerVariant =
-    | 'warning'
-    | 'info'
-    | 'error'
-    | 'success'
-    | 'neutral'
-    | 'secondary';
+type BannerVariant = 'warning' | 'info' | 'error' | 'success';
 
 export interface IMessageBanner {
     message: string;
@@ -66,7 +65,7 @@ export const MessageBanner = ({ messageBanner }: IMessageBannerProps) => {
 
     const {
         message,
-        variant = 'neutral',
+        variant = 'info',
         sticky,
         icon,
         link,
@@ -103,10 +102,8 @@ export const MessageBanner = ({ messageBanner }: IMessageBannerProps) => {
 const VariantIcons = {
     warning: <WarningAmber />,
     info: <InfoOutlined />,
-    error: <Close />,
+    error: <ErrorOutlineRounded />,
     success: <Check />,
-    neutral: <InfoOutlined />,
-    secondary: <InfoOutlined />,
 };
 
 interface IBannerIconProps {