Merge pull request #38 from navikt/appstate-memoized

memoize fns, reset version-state on contentchange

Author: talepre

xp-archive/client/context/appState/AppStateProvider.tsx

@@ -1,4 +1,4 @@
-import React, { useState } from 'react';
+import React, { useCallback, useState } from 'react';
 import { AppStateContext } from './AppStateContext';
 
 type Props = {
@@ -9,13 +9,19 @@ export const AppStateProvider = ({ children }: Props) => {
     const [selectedContentId, setSelectedContentId] = useState<string>();
     const [selectedVersionId, setSelectedVersionId] = useState<string>();
 
+    const updateSelectedContentId = useCallback((selectedContentId: string) => {
+        setSelectedVersionId(undefined);
+        setSelectedContentId(selectedContentId);
+    }, []);
+    const setSelectedVersionIdMemoized = useCallback(setSelectedVersionId, [setSelectedVersionId]);
+
     return (
         <AppStateContext.Provider
             value={{
                 selectedContentId,
-                setSelectedContentId,
+                setSelectedContentId: updateSelectedContentId,
                 selectedVersionId,
-                setSelectedVersionId,
+                setSelectedVersionId: setSelectedVersionIdMemoized,
             }}
         >
             {children}