[docs] Fix the notification display logic (mui#28389)

gpvidal · Sep 16, 2021 · a8eb649 · a8eb649
1 parent ac5b87d
commit a8eb649
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/docs/notifications.json b/docs/notifications.json
@@ -11,6 +11,6 @@
   },
   {
     "id": 54,
-    "text": "<a style=\"color: inherit;\" target=\"_blank\" rel=\"noopener\" href=\"https://twitter.com/MaterialUI/status/1438518915236126723\">MUI Core v5 is out 🎉</a>"
+    "text": "<a style=\"color: inherit;\" target=\"_blank\" rel=\"noopener\" href=\"https://twitter.com/MaterialUI/status/1438518915236126723\">MUI Core v5</a> is out 🎉"
   }
 ]
diff --git a/docs/src/modules/components/Notifications.js b/docs/src/modules/components/Notifications.js
@@ -103,7 +103,7 @@ export default function Notifications() {
         .then((newMessages) => {
           if (active) {
             const seen = getCookie('lastSeenNotification');
-            const lastSeenNotification = seen === '' ? 0 : parseInt(seen, 10);
+            const lastSeenNotification = seen === undefined ? 0 : parseInt(seen, 10);
             setNotifications({
               messages: newMessages || [],
               lastSeen: lastSeenNotification,

diff --git a/docs/src/modules/utils/helpers.ts b/docs/src/modules/utils/helpers.ts
@@ -226,9 +226,7 @@ export function getDependencies(
  * @return The cookie value
  */
 export function getCookie(name: string): string | undefined {
-  // `process.browser` is set by nextjs where we only use `getCookie`
-  // but this file is imported from nodejs scripts so TypeScript complains for that environment.
-  if ((process as any).browser) {
+  if (typeof document !== 'undefined') {
     const value = `; ${document.cookie}`;
     const parts = value.split(`; ${name}=`);
     if (parts.length === 2) {