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 BlurLayout#setAlpha(float) & Closes #41 #42

Closed
wants to merge 1 commit into from

Conversation

fython
Copy link

@fython fython commented Oct 20, 2018

Fixed bug by adding mLockedAlpha field to save the actual value of alpha.

I also added an alpha animation demo in sample app module.

Signed-off-by: Fung Gwo <fython@163.com>
@emersoncloud emersoncloud mentioned this pull request Nov 1, 2018
@emersoncloud
Copy link
Contributor

Hey @fython I wanted to follow up on your PR. We are not sure of our future plans for the BlurKit project quite yet. To avoid the need for licensing agreements we aren’t going to merge your changes directly. We have resolved your concerns in our latest PR #45. We really appreciate your contributions as it did help contribute to our PR, so definitely PR going forward as it’s a great way to get changes integrated directly =)

@fython
Copy link
Author

fython commented Nov 6, 2018

@emersoncloud You're welcome. This project helps me a lot and the bug has been fixed in time so I am grateful too.

If we have a chance to contribute code again, you could ask me what you want to know directly, such as license aggrement, etc.

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