minor cleanup

packages/element/src/delta.ts

@@ -2057,7 +2057,6 @@ export class ElementsDelta implements DeltaContainer<SceneElementsMap> {
   }
 
   private static stripIrrelevantProps(partial: ElementPartial): ElementPartial {
-    // ElementPartial already excludes id, updated, seed; defensively strip if present
     const {
       id: _id,
       updated: _updated,

packages/element/src/newElement.ts

@@ -455,10 +455,6 @@ export const computeStickyNoteFontSize = (
   text: string,
   element: ExcalidrawTextElement,
   container: ExcalidrawTextContainer,
-  /**
-   * Upper cap to which the font size is allowed to grow back during this
-   * editing session (snapped). If not provided defaults to global max.
-   */
   maxGrowFontSize?: number,
 ): StickyNoteFontComputationResult => {
   const step = STICKY_NOTE_FONT_STEP;
@@ -468,10 +464,7 @@ export const computeStickyNoteFontSize = (
   const snap = (size: number) =>
     Math.max(
       STICKY_NOTE_MIN_FONT_SIZE,
-      Math.min(
-        STICKY_NOTE_MAX_FONT_SIZE,
       Math.max(step, Math.floor(size / step) * step),
-      ),
     );
 
   let size = snap(element.fontSize);
@@ -548,8 +541,6 @@ export const refreshTextDimensions = (
         : textElement.width,
     );
   }
-  // NOTE: sticky note font auto-sizing is handled during editing (WYSIWYG)
-  // so we keep refresh logic unchanged to avoid side-effects elsewhere.
   const dimensions = getAdjustedDimensions(textElement, elementsMap, text);
   return { text, ...dimensions };
 };

packages/excalidraw/data/restore.ts

@@ -185,7 +185,8 @@ const restoreElementWithProperties = <
   element: T,
   extra: Pick<
     T,
-    // @ts-ignore
+    // This extra Pick<T, keyof K> ensure no excess properties are passed.
+    // @ts-ignore TS complains here but type checks the call sites fine.
     keyof K
   > &
     Partial<Pick<ExcalidrawElement, "type" | "x" | "y" | "customData">>,
@@ -195,6 +196,8 @@ const restoreElementWithProperties = <
   const base: Pick<T, keyof ExcalidrawElement> = {
     type: nextType,
     version: element.version || 1,
+    // all elements must have version > 0 so getSceneVersion() will pick up
+    // newly added elements
     versionNonce: element.versionNonce ?? 0,
     index: element.index ?? null,
     isDeleted: element.isDeleted ?? false,
@@ -248,12 +251,16 @@ const restoreElementWithProperties = <
   }
 
   const ret = {
+    // spread the original element properties to not lose unknown ones
+    // for forward-compatibility
     ...element,
+    // normalized properties
     ...base,
     ...getNormalizedDimensions(base),
     ...extra,
   } as unknown as T;
 
+  // strip legacy props (migrated in previous steps)
   delete (ret as any).strokeSharpness;
   delete (ret as any).boundElementIds;
 

packages/excalidraw/wysiwyg/textWysiwyg.tsx

@@ -131,7 +131,6 @@ export const textWysiwyg = ({
     return false;
   };
 
-  // Sticky note: remember initial font size for this edit session
   let stickyNoteInitialFontSize: number | null = null;
 
   const updateWysiwygStyle = () => {