Skip to content

fix: keep focused properties in sync with selection - #53

Open
BasicallyPolaris wants to merge 1 commit into
mainfrom
fix/issue-48-focus-properties
Open

fix: keep focused properties in sync with selection#53
BasicallyPolaris wants to merge 1 commit into
mainfrom
fix/issue-48-focus-properties

Conversation

@BasicallyPolaris

@BasicallyPolaris BasicallyPolaris commented Jul 16, 2026

Copy link
Copy Markdown
Owner

Closes #48

Summary

  • model focused property editing as a transient Zustand edit session instead of pausing and manually rewriting undo history
  • derive controlled input values and canvas/ruler previews from the active session, with explicit commit and Escape discard behavior
  • centralize persistent mutations so active edits commit before document actions
  • use preview geometry for drag, resize, marquee, drawing, and context-menu interactions
  • keep dirty tracking and export persistence aware of active previews

Verification

  • bun run test — 305 tests passed
  • bun run build — passed
  • two-axis standards/spec review — no remaining findings

@BasicallyPolaris
BasicallyPolaris force-pushed the fix/issue-48-focus-properties branch from d44b750 to e786bc9 Compare July 16, 2026 23:47
@BasicallyPolaris
BasicallyPolaris force-pushed the fix/issue-48-focus-properties branch from e786bc9 to 72ae4e6 Compare July 17, 2026 00:33
@BasicallyPolaris BasicallyPolaris changed the title fix: refresh properties when selection changes fix: keep focused properties in sync with selection Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: Small focus issue

1 participant