fix: row height / column width value not applied when pressing Enter in context menu

[bynkook]

PR: Fix row height / column width input not applying value on Enter key

Summary

In the context menu, the "Set row height" and "Set column width" inputs stop
propagation of all keydown events. This means pressing Enter after typing
a value (or adjusting with the spinner) never triggers the parent Menu's
onClick handler, so the value is never applied.

The only way the value was ever applied was by clicking the plain-text label
next to the input (e.g. the "row", "height", "column", "width", or "px"
text nodes) — a hidden and completely unintuitive interaction that most
users will never discover.

---

Reproduction

1. Select one or more rows by clicking the row header.
2. Right-click → "Set row height" (set-row-height).
3. Type a new value in the number input (e.g. 50) and press Enter.
4. Expected: row height updates to 50 px.
5. Actual: nothing happens. The value is ignored.

Same steps apply to "Set column width" (set-column-width).

---

Root Cause

packages/react/src/components/ContextMenu/index.tsx

<input
  onClick={(e) => e.stopPropagation()}
  onKeyDown={(e) => e.stopPropagation()}   // ← blocks ALL keydown, including Enter
  type="number"
  ...
/>

The Menu wrapper component relies on its own onClick to read the input
value and call api.setRowHeight / api.setColumnWidth:

<Menu
  onClick={(e, container) => {
    const targetRowHeight = container.querySelector("input")?.value;
    // applies value …
  }}
>
  …
  <input onKeyDown={(e) => e.stopPropagation()} … />
  …
</Menu>

Because stopPropagation() is called unconditionally, the Enter key never
bubbles up to trigger Menu's onClick. The input's native spinner (↑↓
buttons) also fires only on the input element, so clicking the spinner
likewise never applies the value.

The same pattern exists in insert-row and insert-column inputs.

---

Fix

Allow Enter in the onKeyDown handler to programmatically invoke the
parent menu item's click, which triggers the existing apply logic:

packages/react/src/components/ContextMenu/index.tsx

 // set-row-height input
-onKeyDown={(e) => e.stopPropagation()}
+onKeyDown={(e) => {
+  e.stopPropagation();
+  if (e.key === "Enter") {
+    (e.currentTarget.closest(".luckysheet-cols-menuitem") as HTMLElement)?.click();
+  }
+}}

 // set-column-width input  (identical change)
-onKeyDown={(e) => e.stopPropagation()}
+onKeyDown={(e) => {
+  e.stopPropagation();
+  if (e.key === "Enter") {
+    (e.currentTarget.closest(".luckysheet-cols-menuitem") as HTMLElement)?.click();
+  }
+}}

Note: The same fix should be considered for insert-row and
insert-column inputs, which share the same pattern.

---

Test Steps

1. Select rows/columns via header click.
2. Right-click → "Set row height" or "Set column width".
3. Type a value and press Enter → value should be applied immediately.
4. Use the spinner (↑↓) to change the value, then press Enter → applied.
5. Existing behavior (clicking the text label) should still work unchanged.

---

Affected Files

• packages/react/src/components/ContextMenu/index.tsx

Affected Versions

Confirmed on @fortune-sheet/react@1.0.4 (latest at time of writing).

---

Notes

The spinner click issue (value not applied when clicking ↑↓ without pressing
Enter) has the same root cause: the spinner fires a change event on the
input but this never reaches the Menu's onClick. A more complete fix
would add an onChange handler to the input that immediately invokes the
parent click — but that is a separate, larger change left as a follow-up.