From the independent review of PR #54. The stitch Length control became an InputNumber whose onChange calls updateSelection (which pushHistorys) on every keystroke (src/views/EditorView.tsx), so typing "45" can create separate undo entries for intermediate values. The previous Slider coalesced into one entry.
Acceptance
- Editing the length produces a single undo entry per committed change (e.g. commit on
onBlur/onPressEnter/onStep, or reuse the existing live-update coalescing path liveUpdateSelection/endLive in src/core/store.ts).
Severity: minor undo-granularity nit. (Note: this also gives liveUpdateSelection/endLive an in-app caller again — they're currently exercised only by tests.)
From the independent review of PR #54. The stitch Length control became an
InputNumberwhoseonChangecallsupdateSelection(whichpushHistorys) on every keystroke (src/views/EditorView.tsx), so typing "45" can create separate undo entries for intermediate values. The previous Slider coalesced into one entry.Acceptance
onBlur/onPressEnter/onStep, or reuse the existing live-update coalescing pathliveUpdateSelection/endLiveinsrc/core/store.ts).Severity: minor undo-granularity nit. (Note: this also gives
liveUpdateSelection/endLivean in-app caller again — they're currently exercised only by tests.)