box-annotations 4.1.1-beta.5 → 5.1.0

package/src/popup/PopupManager.tsx

@@ -1,5 +1,5 @@
 import * as React from 'react';
-import * as ReactDOM from 'react-dom';
+import * as ReactDOM from 'react-dom/client';
 import BaseManager, { Props } from '../common/BaseManager';
 import PopupContainer from './PopupContainer';
 
@@ -10,6 +10,10 @@ export default class PopupManager extends BaseManager {
     }
 
     render(props: Props): void {
-        ReactDOM.render(<PopupContainer location={this.location} {...props} />, this.reactEl);
+        if (!this.root) {
+            this.root = ReactDOM.createRoot(this.reactEl);
+        }
+
+        this.root.render(<PopupContainer location={this.location} {...props} />);
     }
 }

package/src/region/RegionCreationManager.tsx

@@ -1,5 +1,5 @@
 import * as React from 'react';
-import * as ReactDOM from 'react-dom';
+import * as ReactDOM from 'react-dom/client';
 import BaseManager, { Props } from '../common/BaseManager';
 import RegionCreationContainer from './RegionCreationContainer';
 
@@ -10,6 +10,10 @@ export default class RegionManager extends BaseManager {
     }
 
     render(props: Props): void {
-        ReactDOM.render(<RegionCreationContainer location={this.location} {...props} />, this.reactEl);
+        if (!this.root) {
+            this.root = ReactDOM.createRoot(this.reactEl);
+        }
+
+        this.root.render(<RegionCreationContainer location={this.location} {...props} />);
     }
 }

package/src/region/RegionManager.tsx

@@ -1,5 +1,5 @@
 import * as React from 'react';
-import * as ReactDOM from 'react-dom';
+import * as ReactDOM from 'react-dom/client';
 import BaseManager, { Props } from '../common/BaseManager';
 import RegionAnnotationsContainer from './RegionAnnotationsContainer';
 
@@ -10,6 +10,10 @@ export default class RegionListManager extends BaseManager {
     }
 
     render(props: Props): void {
-        ReactDOM.render(<RegionAnnotationsContainer location={this.location} {...props} />, this.reactEl);
+        if (!this.root) {
+            this.root = ReactDOM.createRoot(this.reactEl);
+        }
+
+        this.root.render(<RegionAnnotationsContainer location={this.location} {...props} />);
     }
 }

package/src/region/actions.ts

@@ -17,7 +17,6 @@ export const createRegionAction = (arg: CreateArg) => (dispatch: AppThunkDispatc
             message,
             type: 'reply' as const,
         },
-        // eslint-disable-next-line @typescript-eslint/camelcase
         file_version: {
             id: getFileVersionId(state),
         },

package/src/store/highlight/actions.ts

@@ -19,7 +19,7 @@ declare global {
         createNodeIterator(
             root: Node,
             whatToShow?: number,
-            filter?: NodeFilter | Function,
+            filter?: NodeFilter | ((node: Node) => number | null),
             entityReferenceExpansion?: boolean,
         ): NodeIterator;
     }

package/src/store/users/actions.ts

@@ -20,9 +20,9 @@ export const fetchCollaboratorsAction = createAsyncThunk<APICollection<Collabora
         // Wrap the client request in a promise to allow it to be returned and cancelled
         return new Promise<APICollection<Collaborator>>((resolve, reject) => {
             client.getFileCollaborators(fileId, resolve, reject, {
-                filter_term: searchString, // eslint-disable-line @typescript-eslint/camelcase
-                include_groups: false, // eslint-disable-line @typescript-eslint/camelcase
-                include_uploader_collabs: false, // eslint-disable-line @typescript-eslint/camelcase
+                filter_term: searchString,
+                include_groups: false,
+                include_uploader_collabs: false,
             });
         });
     },

package/src/common/usePrevious.ts

@@ -1,13 +0,0 @@
-import React from 'react';
-
-// Returns the last provided value, then caches the new value via an effect
-export default function usePrevious<T>(value: T): T | undefined {
-    const ref = React.useRef<T>();
-
-    // The useEffect callback is only invoked after the parent component has rendered
-    React.useEffect(() => {
-        ref.current = value;
-    }, [value]);
-
-    return ref.current;
-}