Skip to content

Allow deletion of whole words with CTRL+BACKSPACE#1680

Merged
cameronwhite merged 1 commit into
PintaProject:masterfrom
stefan-dangl:dev
Aug 23, 2025
Merged

Allow deletion of whole words with CTRL+BACKSPACE#1680
cameronwhite merged 1 commit into
PintaProject:masterfrom
stefan-dangl:dev

Conversation

@stefan-dangl

Copy link
Copy Markdown
Contributor

I like to use Pinta as a digital white board for brain storming. However, it always annoys me that I can't delete text with CTRL+BACKSPACE like I can do in (almost) every other text editor. That's why I added it.

@stefan-dangl stefan-dangl force-pushed the dev branch 2 times, most recently from 135725e to bfda8a3 Compare August 21, 2025 20:58

@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.

This looks great, thanks for contributing!

The CI builds just had an error for code formatting (see other comment) but it looks good otherwise to merge 👍


public void PerformBackspace (bool control)
{
if (HasSelection ()){

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.

I think adding a space here before the curly brace will fix the formatting errors (you can also just run dotnet format

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh, you are right. Thanks. Format should be fixed now.

@cameronwhite

Copy link
Copy Markdown
Member

Thanks!

@cameronwhite cameronwhite merged commit 4c4bfc1 into PintaProject:master Aug 23, 2025
6 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.

2 participants