1.13.2 release

karamem0 · Aug 1, 2024 · f201217 · f201217
1 parent 3c6b5f7
commit f201217
Show file tree

Hide file tree

Showing 57 changed files with 1,211 additions and 878 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -47,6 +47,7 @@
           "useContext",
           "useError",
           "useMsal",
+          "useRoute",
           "useStore",
           "useService",
           "useReducer",

diff --git a/.github/workflows/workflow.yml b/.github/workflows/workflow.yml
@@ -5,7 +5,7 @@ on:
       - develop
 
 env:
-  BUILD_VERSION: '1.13.1'
+  BUILD_VERSION: '1.13.2'
 
 jobs:
   select-environment:

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -20,73 +20,74 @@
   "dependencies": {
     "@automapper/core": "^8.8.1",
     "@automapper/pojos": "^8.8.1",
-    "@azure/msal-browser": "^3.14.0",
+    "@azure/msal-browser": "^3.20.0",
     "@azure/msal-react": "^2.0.22",
-    "@fluentui/react-components": "^9.51.1",
+    "@fluentui/react-components": "^9.54.7",
     "@fluentui/react-icons-mdl2": "^1.3.72",
     "@fluentui/react-icons-mdl2-branded": "^1.2.74",
-    "@microsoft/applicationinsights-react-js": "^17.2.0",
-    "@microsoft/applicationinsights-web": "^3.2.2",
+    "@microsoft/applicationinsights-react-js": "^17.3.0",
+    "@microsoft/applicationinsights-web": "^3.3.0",
     "@microsoft/microsoft-graph-client": "^3.0.7",
     "bytes": "^3.1.2",
     "mime": "^4.0.4",
-    "monaco-editor": "^0.48.0",
+    "monaco-editor": "^0.50.0",
     "react": "^18.3.1",
     "react-dom": "^18.3.1",
     "react-dropzone": "^14.2.3",
     "react-error-boundary": "^4.0.13",
-    "react-hook-form": "^7.51.5",
+    "react-hook-form": "^7.52.1",
     "react-intl": "^6.6.8",
     "react-markdown": "^9.0.1",
-    "react-router-dom": "^6.23.1",
+    "react-router-dom": "^6.25.1",
     "react-syntax-highlighter": "^15.5.0",
     "react-use": "^17.5.1",
     "rehype-raw": "^7.0.0",
     "remark-gfm": "^4.0.0",
     "ress": "^5.0.2"
   },
   "devDependencies": {
-    "@babel/core": "^7.24.9",
-    "@babel/preset-env": "^7.24.8",
+    "@babel/core": "^7.25.2",
+    "@babel/preset-env": "^7.25.2",
     "@babel/preset-react": "^7.24.7",
     "@babel/preset-typescript": "^7.24.7",
-    "@emotion/babel-plugin": "^11.11.0",
-    "@emotion/jest": "^11.11.0",
-    "@emotion/react": "^11.11.4",
+    "@emotion/babel-plugin": "^11.12.0",
+    "@emotion/jest": "^11.13.0",
+    "@emotion/react": "^11.13.0",
     "@formatjs/cli": "^6.2.12",
     "@microsoft/microsoft-graph-types": "^2.40.0",
     "@testing-library/jest-dom": "^6.4.8",
     "@testing-library/react": "^15.0.7",
+    "@testing-library/user-event": "^14.5.2",
     "@types/bytes": "^3.1.4",
     "@types/jest": "^29.5.12",
     "@types/mime": "^3.0.4",
     "@types/react": "^18.3.3",
     "@types/react-dom": "^18.3.0",
     "@types/react-syntax-highlighter": "^15.5.13",
-    "@typescript-eslint/eslint-plugin": "^7.9.0",
-    "@typescript-eslint/parser": "^7.9.0",
-    "@vitejs/plugin-react": "^4.2.1",
+    "@typescript-eslint/eslint-plugin": "^7.18.0",
+    "@typescript-eslint/parser": "^7.18.0",
+    "@vitejs/plugin-react": "^4.3.1",
     "babel-jest": "^29.7.0",
     "babel-plugin-formatjs": "^10.5.16",
     "eslint": "^8.57.0",
     "eslint-config-standard": "^17.1.0",
     "eslint-plugin-hooks": "^0.4.3",
     "eslint-plugin-import": "^2.29.1",
-    "eslint-plugin-react": "^7.34.4",
+    "eslint-plugin-react": "^7.35.0",
     "eslint-plugin-react-hooks": "^4.6.2",
     "eslint-plugin-sonarjs": "^1.0.4",
     "jest": "^29.7.0",
     "jest-environment-jsdom": "^29.7.0",
     "jest-junit": "^16.0.0",
-    "postcss": "^8.4.39",
+    "postcss": "^8.4.40",
     "postcss-scss": "^4.0.9",
     "postcss-styled-syntax": "^0.6.4",
     "sass": "^1.77.8",
-    "stylelint": "^16.5.0",
+    "stylelint": "^16.8.1",
     "stylelint-config-recess-order": "^5.0.1",
     "stylelint-config-standard-scss": "^13.1.0",
-    "typescript": "^5.4.5",
-    "vite": "^5.2.13"
+    "typescript": "^5.5.4",
+    "vite": "^5.3.5"
   },
   "volta": {
     "node": "20.11.0"

diff --git a/src/common/components/Communication.presenter.tsx b/src/common/components/Communication.presenter.tsx
@@ -37,6 +37,7 @@ function Communication(props: Readonly<CommunicationProps>) {
         grid-template-rows: auto auto;
         grid-template-columns: auto;
         grid-gap: 2rem;
+        padding: 1rem;
       `}>
       <img
         alt={title}
@@ -56,6 +57,7 @@ function Communication(props: Readonly<CommunicationProps>) {
           color: ${theme.colorNeutralForeground4};
         `}>
         <Text
+          as="h4"
           css={css`
             font-size: ${theme.fontSizeHero800};
             line-height: calc(${theme.fontSizeHero800} * 1.25);

diff --git a/src/common/components/ScrollPanel.tsx b/src/common/components/ScrollPanel.tsx
@@ -41,7 +41,6 @@ function ScrollPanel(props: Readonly<ScrollPanelProps>) {
   } = props;
 
   const [ state, setState ] = React.useState<ScrollPanelState>({});
-
   const ref = React.useRef<HTMLDivElement>(null);
 
   const handleKeyDown = React.useCallback((event: KeyboardEvent) => {

diff --git a/src/common/components/ScrollSynchronizer.tsx b/src/common/components/ScrollSynchronizer.tsx
@@ -43,7 +43,6 @@ function ScrollSynchronizer(props: Readonly<ScrollSynchronizerProps>) {
 
   const [ element1Position, setElement1Position ] = React.useState<ScrollPosition | undefined>(defaultElement1Position);
   const [ element2Position, setElement2Position ] = React.useState<ScrollPosition | undefined>(defaultElement2Position);
-
   const element1Active = React.useRef<boolean>();
   const element1Size = React.useRef<ScrollSize>();
   const element2Active = React.useRef<boolean>();

diff --git a/src/common/components/SidePanel.presenter.tsx b/src/common/components/SidePanel.presenter.tsx
@@ -43,7 +43,7 @@ function SidePanel(props: Readonly<React.PropsWithChildren<SidePanelProps>>) {
       position="end"
       size="medium"
       css={css`
-        height: 100vh;
+        height: 100svh;
       `}
       onOpenChange={(event, data) => onOpenChange?.(event, data.open)}>
       <DrawerHeader>

diff --git a/src/common/components/Tree.tsx b/src/common/components/Tree.tsx
@@ -9,6 +9,7 @@
 import React from 'react';
 
 import { Event } from '../../types/Event';
+import { moveNext, movePrevious } from '../../utils/Keyboard';
 
 import Presenter from './Tree.presenter';
 
@@ -29,31 +30,11 @@ function Tree(props: Readonly<React.PropsWithChildren<TreeProps>>) {
     const { key } = event as KeyboardEvent;
     switch (key) {
       case 'ArrowDown': {
-        const { current: element } = ref;
-        const items = element?.querySelectorAll<HTMLDivElement>('div[role="menuitem"]');
-        if (items == null) {
-          break;
-        }
-        for (let i = 0; i < items?.length - 1; i++) {
-          if (items[i] === document.activeElement) {
-            items[i + 1].focus();
-            break;
-          }
-        }
+        moveNext(ref.current, 'div[role="menuitem"]');
         break;
       }
       case 'ArrowUp': {
-        const { current: element } = ref;
-        const items = element?.querySelectorAll<HTMLDivElement>('div[role="menuitem"]');
-        if (items == null) {
-          break;
-        }
-        for (let i = 1; i < items?.length; i++) {
-          if (items[i] === document.activeElement) {
-            items[i - 1].focus();
-            break;
-          }
-        }
+        movePrevious(ref.current, 'div[role="menuitem"]');
         break;
       }
       default:

diff --git a/src/common/components/TreeItem.presenter.tsx b/src/common/components/TreeItem.presenter.tsx
@@ -71,7 +71,8 @@ function TreeItem(props: Readonly<TreeItemProps>) {
           background-color: ${theme.colorNeutralBackground2Hover};
         }
       `}
-      onKeyDown={onKeyDown}>
+      onKeyDown={onKeyDown}
+      onTouchStart={onClick}>
       <div
         css={css`
           display: grid;

diff --git a/src/common/components/VideoViewer.tsx b/src/common/components/VideoViewer.tsx
@@ -23,7 +23,6 @@ function VideoViewer(props: Readonly<VideoViewerProps>) {
   } = props;
 
   const [ loading, setLoading ] = React.useState<boolean>(true);
-
   const ref = React.useRef<HTMLVideoElement>(null);
 
   React.useEffect(() => {

diff --git a/src/features/dialog/components/FileCopyDialog.tsx b/src/features/dialog/components/FileCopyDialog.tsx
@@ -32,9 +32,7 @@ function FileCopyDialog(props: Readonly<FileCopyDialogProps>) {
 
   const { value } = props;
 
-  const {
-    dispatch
-  } = useStore();
+  const { dispatch } = useStore();
   const { graph } = useService();
   const [ loading, setLoading ] = React.useState<boolean>(false);
 
@@ -52,11 +50,7 @@ function FileCopyDialog(props: Readonly<FileCopyDialogProps>) {
       dispatch(setContentFile(file));
       dispatch(setContentText(await graph.getFileText(file)));
     } catch (error) {
-      if (error instanceof Error) {
-        dispatch(setError(error));
-        return;
-      }
-      throw error;
+      dispatch(setError(error as Error));
     } finally {
       setLoading(false);
       dispatch(setDialogAction());

diff --git a/src/features/dialog/components/FileCreateDialog.tsx b/src/features/dialog/components/FileCreateDialog.tsx
@@ -55,11 +55,7 @@ function FileCreateDialog() {
         file: file?.id
       });
     } catch (error) {
-      if (error instanceof Error) {
-        dispatch(setError(error));
-        return;
-      }
-      throw error;
+      dispatch(setError(error as Error));
     } finally {
       setLoading(false);
       dispatch(setDialogAction());

diff --git a/src/features/dialog/components/FileDeleteDialog.tsx b/src/features/dialog/components/FileDeleteDialog.tsx
@@ -65,11 +65,7 @@ function FileDeleteDialog(props: Readonly<FileDeleteDialogProps>) {
         });
       }
     } catch (error) {
-      if (error instanceof Error) {
-        dispatch(setError(error));
-        return;
-      }
-      throw error;
+      dispatch(setError(error as Error));
     } finally {
       setLoading(false);
       dispatch(setDialogAction());

diff --git a/src/features/dialog/components/FileOverwriteDialog.tsx b/src/features/dialog/components/FileOverwriteDialog.tsx
@@ -31,9 +31,7 @@ function FileOverwriteDialog(props: Readonly<FileOverwriteDialogProps>) {
 
   const { value } = props;
 
-  const {
-    dispatch
-  } = useStore();
+  const { dispatch } = useStore();
   const { graph } = useService();
   const [ loading, setLoading ] = React.useState<boolean>(false);
 
@@ -52,10 +50,7 @@ function FileOverwriteDialog(props: Readonly<FileOverwriteDialogProps>) {
       }
       dispatch(removeExplorerFileConflict(value));
     } catch (error) {
-      if (error instanceof Error) {
-        dispatch(setError(error));
-        return;
-      }
+      dispatch(setError(error as Error));
       throw error;
     } finally {
       setLoading(false);

diff --git a/src/features/dialog/components/FileRenameDialog.tsx b/src/features/dialog/components/FileRenameDialog.tsx
@@ -63,18 +63,14 @@ function FileRenameDialog(props: Readonly<FileRenameDialogProps>) {
         }));
       }
     } catch (error) {
-      if (error instanceof Error) {
-        dispatch(setError(error));
-        return;
-      }
-      throw error;
+      dispatch(setError(error as Error));
     } finally {
       setLoading(false);
       dispatch(setDialogAction());
     }
   }, [
-    graph,
     contentProps?.file,
+    graph,
     dispatch
   ]);
 

diff --git a/src/features/dialog/components/FileRestoreDialog.tsx b/src/features/dialog/components/FileRestoreDialog.tsx
@@ -50,19 +50,15 @@ function FileDeleteDialog(props: Readonly<FileRestoreDialogProps>) {
       await graph.restoreFile(value);
       dispatch(setContentText(await graph.getFileText(contentFile)));
     } catch (error) {
-      if (error instanceof Error) {
-        dispatch(setError(error));
-        return;
-      }
-      throw error;
+      dispatch(setError(error as Error));
     } finally {
       setLoading(false);
       dispatch(setDialogAction());
       dispatch(setSidePanelAction());
     }
   }, [
-    graph,
     contentProps?.file,
+    graph,
     value,
     dispatch
   ]);

diff --git a/src/features/dialog/components/FolderCreateDialog.tsx b/src/features/dialog/components/FolderCreateDialog.tsx
@@ -45,11 +45,7 @@ function FolderCreateDialog() {
       const folder = await graph.createFolder(selectedFolder, `${data.name}`);
       dispatch(appendExplorerFolder(folder));
     } catch (error) {
-      if (error instanceof Error) {
-        dispatch(setError(error));
-        return;
-      }
-      throw error;
+      dispatch(setError(error as Error));
     } finally {
       setLoading(false);
       dispatch(setDialogAction());

diff --git a/src/features/dialog/components/FolderDeleteDialog.tsx b/src/features/dialog/components/FolderDeleteDialog.tsx
@@ -28,9 +28,7 @@ function FileDeleteDialog(props: Readonly<FolderDeleteDialogProps>) {
 
   const { value } = props;
 
-  const {
-    dispatch
-  } = useStore();
+  const { dispatch } = useStore();
   const { graph } = useService();
   const [ loading, setLoading ] = React.useState<boolean>(false);
 
@@ -42,11 +40,7 @@ function FileDeleteDialog(props: Readonly<FolderDeleteDialogProps>) {
       await graph.deleteFolder(value);
       dispatch(removeExplorerFolder(value));
     } catch (error) {
-      if (error instanceof Error) {
-        dispatch(setError(error));
-        return;
-      }
-      throw error;
+      dispatch(setError(error as Error));
     } finally {
       setLoading(false);
       dispatch(setDialogAction());

diff --git a/src/features/dialog/components/FolderRenameDialog.tsx b/src/features/dialog/components/FolderRenameDialog.tsx
@@ -46,11 +46,7 @@ function FolderRenameDialog(props: Readonly<FolderRenameDialogProps>) {
       const folder = await graph.renameFolder(data, data.name);
       dispatch(updateExplorerFolder(folder));
     } catch (error) {
-      if (error instanceof Error) {
-        dispatch(setError(error));
-        return;
-      }
-      throw error;
+      dispatch(setError(error as Error));
     } finally {
       setLoading(false);
       dispatch(setDialogAction());