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

new image processors: blend, compositing #818

Merged
merged 1 commit into from
Dec 4, 2017

Conversation

yemodin
Copy link
Contributor

@yemodin yemodin commented Dec 3, 2017

Adds blending/compositing(macOS) processors. Allows easily blend/compositing.

For example, I was needed to blend unedited images which have the different background color so they will appear blended with the background color of my interface.

@yemodin
Copy link
Contributor Author

yemodin commented Dec 3, 2017

Also, I changed macOS version to 10.11 to successfully build and run.

@onevcat
Copy link
Owner

onevcat commented Dec 4, 2017

@yemodin Looks great.

However, if we increase the minimal deploy target, we have to release another major update since it breaks compatibility. May I know which API you are using is 10.11 above?

And do not worry about the failing tests. If we could add this processor to the framework, I will check and fix these tests.

@yemodin
Copy link
Contributor Author

yemodin commented Dec 4, 2017

@onevcat I changed only demo project target, not framework target. Next lines of macOS demo require 10.11+:


let item = collectionView.makeItem(withIdentifier: NSUserInterfaceItemIdentifier(rawValue: "Cell"), for: indexPath)

@yemodin
Copy link
Contributor Author

yemodin commented Dec 4, 2017

Also, I added the test to each image processor. And made the pull request for image assets:
onevcat/Kingfisher-TestImages#1

@onevcat
Copy link
Owner

onevcat commented Dec 4, 2017

Oh, cool! I will check it now.

@onevcat onevcat merged commit 33efa6a into onevcat:master Dec 4, 2017
petarlazarov pushed a commit to dowjones-mobile/Kingfisher that referenced this pull request Oct 6, 2021
new image processors: blend, compositing
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