Skip to content
New issue

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

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

Already on GitHub? # to your account

Add alpha erase blend mode #1117

Merged
merged 2 commits into from
Oct 12, 2024

Conversation

Variable-ind
Copy link
Contributor

Works same as the Erase mode in https://docs.gimp.org/en/gimp-concepts-layer-modes.html (Figure 8.12. Example for layer mode “Erase”)

An "Erase" layer blend mode or effect would easily skip a step of manual labor when, for example, making opaque shapes with transparent icons within.

effect-.2.mp4
effect-.2.mp4

@OverloadedOrama
Copy link
Member

Looks good! If we want to be consistent with other software though, I think we should keep the name "Subtract" as it is, and the new mode should be called "Erase", as they are not quite the same. Erase should also be together with Normal, as its not considered an Inversion type of blend mode. In the documentation you linked it is listed under the Normal layer modes group, while Subtract is listed under the Inversion layer modes https://docs.gimp.org/en/layer-mode-group-inversion.html

@Variable-ind
Copy link
Contributor Author

Done

Copy link
Member

@OverloadedOrama OverloadedOrama left a comment

Choose a reason for hiding this comment

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

Looks great, thank you!

@OverloadedOrama OverloadedOrama merged commit c21e089 into Orama-Interactive:master Oct 12, 2024
3 of 4 checks passed
@Variable-ind Variable-ind deleted the erase-blend branch October 13, 2024 01:38
OverloadedOrama added a commit that referenced this pull request Oct 22, 2024
… pxo files

The addition of the erase blend mode from #1117 resulted in loading pxo files from v1.0-v1.0.3 to have incorrect blend modes in their layers, if they are set to anything below normal, because the values of the `BaseLayer.BlendModes` enumerator changed.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants