Merge pull request #5961 from WiXSL/fix-success-notification-type

Add success notification type
marmelab · Feb 26, 2021 · 037af98 · 037af98
2 parents e989299 + 7141a2a
commit 037af98
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/packages/ra-core/src/actions/notificationActions.ts b/packages/ra-core/src/actions/notificationActions.ts
@@ -1,6 +1,6 @@
 export const SHOW_NOTIFICATION = 'RA/SHOW_NOTIFICATION';
 
-export type NotificationType = 'info' | 'warning' | 'error';
+export type NotificationType = 'success' | 'info' | 'warning' | 'error';
 
 interface NotificationOptions {
     // The duration in milliseconds the notification is shown

diff --git a/packages/ra-ui-materialui/src/layout/Notification.tsx b/packages/ra-ui-materialui/src/layout/Notification.tsx
@@ -22,6 +22,10 @@ interface Props {
 
 const useStyles = makeStyles(
     (theme: Theme) => ({
+        success: {
+            backgroundColor: theme.palette.success.main,
+            color: theme.palette.success.contrastText,
+        },
         error: {
             backgroundColor: theme.palette.error.dark,
             color: theme.palette.error.contrastText,