Merge pull request #1092 from equinor/master

Release: Bugfix rerendering NAV (#1091)
equinor · Sep 18, 2024 · e1051b9 · e1051b9
2 parents 68e55c3 + adaa3b8
commit e1051b9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/components/app-navbar/index.tsx b/src/components/app-navbar/index.tsx
@@ -121,8 +121,8 @@ const NavbarExpanded = ({ appName, links }: NavbarProps) => {
         </Tooltip>
       </span>
 
-      {links.map((link, i) => (
-        <NavbarLink key={i} {...link} />
+      {links.map((link) => (
+        <NavbarLink key={link.to} {...link} />
       ))}
 
       <Typography

diff --git a/src/effects/use-local-storage.ts b/src/effects/use-local-storage.ts
@@ -35,7 +35,8 @@ export default function useLocalStorage<T>(
 
   useInterval(() => {
     const current = getLocalStorageItem(key);
-    if (current != state) {
+    if (JSON.stringify(current) != JSON.stringify(state)) {
+      console.log('updating state', { current, state });
       setState(current);
     }
   }, 250);