Skip to content

feat(PaintWidget2D): Add modifier+scroll shortcut for brush size#765

Merged
floryst merged 3 commits intoKitware:mainfrom
TomTonroe:brush-size-shortcut
Jul 25, 2025
Merged

feat(PaintWidget2D): Add modifier+scroll shortcut for brush size#765
floryst merged 3 commits intoKitware:mainfrom
TomTonroe:brush-size-shortcut

Conversation

@TomTonroe
Copy link
Contributor

Adds Ctrl+scroll (mouse wheel or two finger scroll) functionality to adjust paint brush size.

Related to #716!

@netlify
Copy link

netlify bot commented Jul 13, 2025

Deploy Preview for volview-dev ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit f2e36a0
🔍 Latest deploy log https://app.netlify.com/projects/volview-dev/deploys/6877239314d5cd0008acc014
😎 Deploy Preview https://deploy-preview-765--volview-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@PaulHax
Copy link
Collaborator

PaulHax commented Jul 15, 2025

Really nice. If we add this behavior to the "actions", folks can customaize the key from ctrl to whatever and get a little help docs with the ? key.

https://github.com/Kitware/VolView/pull/743/files#diff-e1d24894ec9819345d29797efc24216217745493ebf91cfe71ffd339d4ffbc33R87-L89

@TomTonroe TomTonroe changed the title feat(PaintWidget2D): Add ctrl+scroll shortcut for brush size feat(PaintWidget2D): Add modifier+scroll shortcut for brush size Jul 16, 2025
@TomTonroe
Copy link
Contributor Author

Thanks! I have added the ability to change the modifier key (ctrl as default) and updated the documentation.

Copy link
Contributor

@floryst floryst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for the contribution! 🎉

@floryst floryst added this pull request to the merge queue Jul 25, 2025
Merged via the queue into Kitware:main with commit 971b364 Jul 25, 2025
7 checks passed
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.

3 participants