Skip to content

Commit e1635fa

Browse files
committed
fix: better to use onInitialize to set ExcalidrawImperativeAPI
1 parent 49e4cee commit e1635fa

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

src/components/whiteboard/index.tsx

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -550,6 +550,15 @@ const Whiteboard = ({ onReadyExcalidrawAPI }: WhiteboardProps) => {
550550
[excalidrawAPI, isPresenter, isFollowing, showSwitchingWarning],
551551
);
552552

553+
const onInitializeSetExcalidrawAPI = useCallback(
554+
(api: ExcalidrawImperativeAPI) => {
555+
setExcalidrawAPI(api);
556+
onReadyExcalidrawAPI(api);
557+
},
558+
// oxlint-disable-next-line eslint-plugin-react-hooks/exhaustive-deps
559+
[],
560+
);
561+
553562
return (
554563
<div className="excalidraw-wrapper flex-1 w-full max-w-[1140px] m-auto h-[calc(100%-50px)] sm:px-5 mt-9 z-0">
555564
{isPresenter && excalidrawAPI && (
@@ -562,12 +571,7 @@ const Whiteboard = ({ onReadyExcalidrawAPI }: WhiteboardProps) => {
562571
/>
563572
)}
564573
<Excalidraw
565-
onExcalidrawAPI={(api) => {
566-
if (api) {
567-
setExcalidrawAPI(api);
568-
onReadyExcalidrawAPI(api);
569-
}
570-
}}
574+
onInitialize={onInitializeSetExcalidrawAPI}
571575
onChange={handleCanvasChange}
572576
onPointerUpdate={onPointerUpdate}
573577
onScrollChange={onScrollChange}

0 commit comments

Comments
 (0)