-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update blend-modes.md with Screen definition resolve #204 #217
Conversation
Adding blend mode "Screen" definition. Please review.
Adding blend mode "Screen" definition. Please review.
Please link to issue #204 if all looks good. Thank you. |
small indentation fix
docs/en/shared/blend-modes.md
Outdated
@@ -8,3 +8,6 @@ Add | |||
|
|||
Multiply | |||
: Darken the background with values of the the corresponding pixels of the component: `src.rgb * dst.rgb` | |||
|
|||
Screen | |||
: Opposite of Multiply. Brighten background and values of the the corresponding pixels of the component: `src.rgb - dst.rgb * dst.rgb` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"the the"
Oh, I see we have the same on Multiply!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it! removed the extra "the" from both descriptions. I will make sure to proofread a bit more in future pr's !
Removed double wording "the the" in both Multiply and Screen definitions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Adding blend mode "Screen" definition. Please review.