Merge pull request #197 from Azure-Samples/emlyn/update-ui-library-1.28.0

Update to communication-react-1.28.0

Author: emlynmac

Chat/package-lock.json

@@ -17,14 +17,14 @@
     "lint:quiet": "eslint **/*.{ts,tsx} --quiet"
   },
   "dependencies": {
-    "@azure/abort-controller": "^1.1.0",
+    "@azure/abort-controller": "^2.1.0",
     "@azure/communication-chat": "^1.5.4",
     "@azure/communication-common": "^2.3.1",
     "@azure/communication-identity": "^1.3.0",
-    "@azure/communication-react": "1.26.0",
+    "@azure/communication-react": "1.28.0",
     "@azure/logger": "^1.0.4",
     "@fluentui/react": "^8.115.2",
-    "@fluentui/react-file-type-icons": "8.11.4",
+    "@fluentui/react-file-type-icons": "^8.11.4",
     "@fluentui/react-icons": "^2.0.231",
     "@types/react-linkify": "^1.0.4",
     "json-stringify-safe": "^5.0.1",
@@ -60,8 +60,8 @@
     "eslint-plugin-react": "^7.18.3",
     "eslint-plugin-react-hooks": "^4.1.2",
     "html-webpack-plugin": "~5.3.1",
-    "husky": "^4.3.0",
-    "jest": "29.7.0",
+    "husky": "^9.1.7",
+    "jest": "^29.7.0",
     "jest-fetch-mock": "^3.0.3",
     "jest-junit": "~13.0.0",
     "prettier": "2.3.1",
@@ -71,8 +71,8 @@
     "ts-loader": "^8.0.12",
     "typescript": "^4.3.5",
     "url-loader": "~4.1.1",
-    "webpack": "5.95.0",
-    "webpack-cli": "^4.9.2",
-    "webpack-dev-server": "5.2.1"
+    "webpack": "^5.95.0",
+    "webpack-cli": "^5.1.4",
+    "webpack-dev-server": "^5.2.1"
   }
 }

Chat/package.json

@@ -53,6 +53,7 @@ export const ChatHeader = (props: ChatHeaderProps): JSX.Element => {
         onClick={() => props.onEndChat()}
         ariaLabel={leaveString}
         aria-live={'polite'}
+        title={leaveString}
       />
     </Stack>
   );

Chat/src/app/ChatHeader.tsx

@@ -42,6 +42,7 @@ import {
 import { joinThread } from './utils/joinThread';
 import { getEndpointUrl } from './utils/getEndpointUrl';
 import { refreshToken } from './utils/refreshToken';
+
 import {
   getDisplayNameFromLocalStorage,
   localStorageAvailable,
@@ -158,7 +159,7 @@ export default (props: ConfigurationScreenProps): JSX.Element => {
           if (!threadId) {
             throw new Error(ERROR_TEXT_THREAD_NOT_RECORDED);
           }
-        } catch (error) {
+        } catch {
           setConfigurationScreenState(CONFIGURATIONSCREEN_SHOWING_INVALID_THREAD);
           return;
         }

Chat/src/app/ConfigurationScreen.tsx

@@ -51,7 +51,7 @@ const DisplayNameFieldComponent = (props: DisplayNameFieldProps): JSX.Element =>
       placeholder={TEXTFIELD_PLACEHOLDER}
       onKeyDown={(ev) => {
         if (ev.which === ENTER_KEY) {
-          validateName && validateName();
+          validateName?.();
         }
       }}
       styles={TextFieldStyleProps}

Chat/src/app/DisplayNameField.tsx

@@ -1,3 +1 @@
-// Copyright (c) Microsoft Corporation.
-// Licensed under the MIT License.
 export {}; /*The above line is generated by conditional compilation, when no export detected after CC.*/

Chat/src/app/RichTextEditorToggle.tsx

@@ -16,19 +16,19 @@ export const inputBoxStyle = mergeStyles({
 });
 
 export const inputBoxTextStyle = mergeStyles({
-  fontSize: '0.875rem',
+  fontSize: '1rem',
   fontWeight: 600,
   lineHeight: '1.5rem',
   '::-webkit-input-placeholder': {
-    fontSize: '0.875rem',
+    fontSize: '1rem',
     fontWeight: 600
   },
   '::-moz-placeholder': {
-    fontSize: '0.875rem',
+    fontSize: '1rem',
     fontWeight: 600
   },
   ':-moz-placeholder': {
-    fontSize: '0.875rem',
+    fontSize: '1rem',
     fontWeight: 600
   }
 });

Chat/src/app/styles/DisplayNameField.styles.ts

@@ -71,9 +71,9 @@ const defaultTheme: NamedTheme = defaultThemes.Light as NamedTheme;
 const SwitchableFluentThemeContext = createContext<SwitchableFluentThemeContext>({
   currentTheme: defaultTheme,
   currentRtl: false,
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-empty-function
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   setCurrentTheme: (theme: NamedTheme) => {},
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-empty-function
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   setCurrentRtl: (rtl: boolean) => {},
   themeStore: defaultThemes
 });

Chat/src/app/theming/SwitchableFluentThemeProvider.tsx

@@ -1,3 +1 @@
-// Copyright (c) Microsoft Corporation.
-// Licensed under the MIT License.
 export {}; /*The above line is generated by conditional compilation, when no export detected after CC.*/

Chat/src/app/utils/downloadHandler.tsx

@@ -12,22 +12,22 @@ export enum LocalStorageKeys {
  * Get display name from local storage.
  */
 export const getDisplayNameFromLocalStorage = (): string | null =>
-  window.localStorage.getItem(LocalStorageKeys.DisplayName);
+  window.localStorage?.getItem(LocalStorageKeys.DisplayName);
 
 /**
  * Save display name into local storage.
  */
 export const saveDisplayNameToLocalStorage = (displayName: string): void =>
-  window.localStorage.setItem(LocalStorageKeys.DisplayName, displayName);
+  window.localStorage?.setItem(LocalStorageKeys.DisplayName, displayName);
 
 /**
  * Get theme from local storage.
  */
 export const getThemeFromLocalStorage = (scopeId: string): string | null =>
-  window.localStorage.getItem(LocalStorageKeys.Theme + '_' + scopeId);
+  window.localStorage?.getItem(LocalStorageKeys.Theme + '_' + scopeId);
 
 /**
  * Save theme into local storage.
  */
 export const saveThemeToLocalStorage = (theme: string, scopeId: string): void =>
-  window.localStorage.setItem(LocalStorageKeys.Theme + '_' + scopeId, theme);
+  window.localStorage?.setItem(LocalStorageKeys.Theme + '_' + scopeId, theme);