Skip to content

Added "Offset Selection" action#746

Merged
cameronwhite merged 4 commits into
PintaProject:masterfrom
zWolfrost:master
Mar 16, 2024
Merged

Added "Offset Selection" action#746
cameronwhite merged 4 commits into
PintaProject:masterfrom
zWolfrost:master

Conversation

@zWolfrost

Copy link
Copy Markdown
Contributor

Hi, i have added the GUI option for offsetting the selection in the edit tab, finally closing my issue (#661).
Similarly to last time, I have tried my best to respect the project flow, but feel free to point out if i did something wrong.
I will add an icon for the action in a next commit if i manage to find one, but for now i will just use the same icon of the "invert selection" option.

@zWolfrost

Copy link
Copy Markdown
Contributor Author

I have noticed that the code formatting check for Ubuntu wasn't successful, but that should be an unrelated problem and not caused by my pull request. Shouldn't be too bad, the build checks all work anyway.

@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 very good, thank you!

I think the formatting issue might be from having incorrect line endings?

Comment thread Pinta/Actions/Edit/OffsetSelectionAction.cs Outdated
@zWolfrost

zWolfrost commented Mar 15, 2024

Copy link
Copy Markdown
Contributor Author

There you go. So in the end it was caused by my PR, my bad. For future reference, run the command "dotnet format --no-restore --exclude-diagnostics CA1416" to let .NET format your code before submitting.

@cameronwhite

Copy link
Copy Markdown
Member

Looks good, thanks for working on this!

@cameronwhite cameronwhite merged commit 434e4eb into PintaProject:master Mar 16, 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