Skip to content
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

Fix the default blending #560

Merged
merged 3 commits into from
Jan 9, 2020
Merged

Fix the default blending #560

merged 3 commits into from
Jan 9, 2020

Conversation

ryanisaacg
Copy link
Owner

@ryanisaacg ryanisaacg commented Jan 8, 2020

The Default::default of an Option is None, which disables blending. That
line was supposed to enable the default blending mode in golem, which is
a sensible blending default, instead of disabling blending entirely.

Motivation and Context

See #552 (comment)

Checks

  • I have read CONTRIBUTING.md.
  • I have updated CHANGES.md, with [BREAKING] next to all breaking changes
  • I have updated the documentation if necessary

The Default::default of an Option is None, which disables blending. That
line was supposed to enable the default blending mode in golem, which is
a sensible blending default.
@MaulingMonkey
Copy link
Contributor

Confirmed fix 👍

@ryanisaacg ryanisaacg merged commit 53758a2 into master Jan 9, 2020
@ryanisaacg ryanisaacg deleted the add-default-blending branch January 9, 2020 00:37
leod pushed a commit to leod/quicksilver that referenced this pull request Jun 3, 2023
The Default::default of an Option is None, which disables blending. That
line was supposed to enable the default blending mode in golem, which is
a sensible blending default.
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