Skip to content

fix(canvas): honor region bounds when snapping resize - #57

Closed
teo-nex wants to merge 1 commit into
howdeploy:mainfrom
teo-nex:fix/canvas-region-snap-limits-20260923
Closed

teo-nex wants to merge 1 commit into
howdeploy:mainfrom
teo-nex:fix/canvas-region-snap-limits-20260923

Conversation

@teo-nex

@teo-nex teo-nex commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Canvas regions support widths from 360 to 4000 px, but snapping reused the terminal-card limits. With snap enabled, resizing a region to 1900 px silently clamped it to 1600 px; a 380 px west-edge resize jumped to 420 px and shifted the opposite edge.

This passes the existing region min/max values into the snap calculation and keeps the west-edge anchor stable. It changes only region resizing.

Validation: 8 focused snap/region tests, 659/659 full project tests, TypeScript typecheck, and git diff --check pass. Live pointer interaction in the packaged app was not exercised.

@howdeploy

Copy link
Copy Markdown
Owner

Integrated into main via #73. The canvas-region fix and your authored commit are preserved in the merge history. Thank you, @teo-nex.

@howdeploy howdeploy closed this Sep 24, 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.

2 participants