Skip to content

Add keyboard shortcut for changing the brush size or line width#1155

Merged
cameronwhite merged 3 commits into
PintaProject:masterfrom
solarnomad7:master
Nov 22, 2024
Merged

Add keyboard shortcut for changing the brush size or line width#1155
cameronwhite merged 3 commits into
PintaProject:masterfrom
solarnomad7:master

Conversation

@solarnomad7

@solarnomad7 solarnomad7 commented Nov 20, 2024

Copy link
Copy Markdown
Contributor

Implements issue #796. The square brackets ] and [ increase and decrease the brush size respectively. Applies to all tools that implement BaseBrushTool and ShapeTool.

I'm wondering if it's possible to add a tooltip to the + and - buttons on the brush size widget? As of right now there's no indication that the shortcuts exist.

@cameronwhite cameronwhite left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for looking at this!
The CI builds failed from some formatting issues - you should be able to just run dotnet format to fix those

For adding a tooltip, I think you should be able to set the TooltipText property on the spin button widget when it's created - this would be a tooltip for the whole widget, not individually for the +/- buttons.
For that string, you would also want to use Translations.GetString("...") so that it can be translated

Comment thread Pinta.Tools/Editable/EditEngines/BaseEditEngine.cs Outdated
@cameronwhite

Copy link
Copy Markdown
Member

Looks good - thanks for your contribution!

@cameronwhite cameronwhite merged commit 280d2fd into PintaProject:master Nov 22, 2024
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