From 19b03b4ca9d9dbbd8d3246d99490dadf57e4c507 Mon Sep 17 00:00:00 2001 From: Omar Brikaa Date: Fri, 10 Oct 2025 19:12:08 +0300 Subject: [PATCH] fix: remove redundant selectionStart/End resetting that causes scroll-reset bug on firefox (#8263) Remove redundant selectionStart/End resetting that causes scroll-reset bug on firefox --- packages/excalidraw/wysiwyg/textWysiwyg.tsx | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/packages/excalidraw/wysiwyg/textWysiwyg.tsx b/packages/excalidraw/wysiwyg/textWysiwyg.tsx index adede07f87..149faf8987 100644 --- a/packages/excalidraw/wysiwyg/textWysiwyg.tsx +++ b/packages/excalidraw/wysiwyg/textWysiwyg.tsx @@ -226,22 +226,6 @@ export const textWysiwyg = ({ } } const [viewportX, viewportY] = getViewportCoords(coordX, coordY); - const initialSelectionStart = editable.selectionStart; - const initialSelectionEnd = editable.selectionEnd; - const initialLength = editable.value.length; - - // restore cursor position after value updated so it doesn't - // go to the end of text when container auto expanded - if ( - initialSelectionStart === initialSelectionEnd && - initialSelectionEnd !== initialLength - ) { - // get diff between length and selection end and shift - // the cursor by "diff" times to position correctly - const diff = initialLength - initialSelectionEnd; - editable.selectionStart = editable.value.length - diff; - editable.selectionEnd = editable.value.length - diff; - } if (!container) { maxWidth = (appState.width - 8 - viewportX) / appState.zoom.value;