Skip to content

Add Snap To Grid to Canvas Toolbar + Hotkey #8205

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

blessedcoolant
Copy link
Collaborator

@blessedcoolant blessedcoolant commented Jul 6, 2025

Summary

Snap to Grid is one of the most commonly used features on the Canvas. Especially during fine editing and refining. Makes no sense to hide away the option to toggle it in the menu (will still be there too because it is a grid setting).

Added it to the primary canvas toolbar now along with the other bounding box tools so it is in a relevant place. Also consistent with other applications that do expose snapping tools out on the main UI.

opera_1AwolyUGkR

Hotkey

Use Shift+S as a new hotkey to quickly enable and disable snapping.

Other

Some other hotkey localization strings were missing. Added them in.

Merge Plan

YOLO

@github-actions github-actions bot added the frontend PRs that change frontend files label Jul 6, 2025
@hipsterusername
Copy link
Member

Is this better/more commonly used than the Ctrl key?

@blessedcoolant
Copy link
Collaborator Author

blessedcoolant commented Jul 7, 2025

Is this better/more commonly used than the Ctrl key?

I avoided the Ctrl S coz it generally binds to saving the HTML page on browsers and that's an unnecessary conflict. But can change it up if it feels better that way.

@psychedelicious
Copy link
Collaborator

You can hold ctrl/cmd while moving bbox or a layer to disable snap

@blessedcoolant
Copy link
Collaborator Author

blessedcoolant commented Jul 7, 2025

You can hold ctrl/cmd while moving bbox or a layer to disable snap

Did not know this was a thing. I don't like you with your facts that question my doc reading ability.

BUT I'd still make the argument that the toggle and the visible indicator on the UI is a good addition. If I could not spot it, it's likely many others wouldn't. Which in itself makes the case for ease of use imo. Also a full toggle and temp toggle both need to be quick access features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend PRs that change frontend files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants