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

Colorize blur #2

Closed
wants to merge 3 commits into from
Closed

Colorize blur #2

wants to merge 3 commits into from

Conversation

imaNNeo
Copy link
Contributor

@imaNNeo imaNNeo commented Jan 13, 2018

added shadow_color attribute and also added setShadowColor and getShadowColor.
tested as well

@vipulasri
Copy link
Owner

vipulasri commented Jan 14, 2018

Hello, @imaNNeoFighT I intentionally didn't implement the colored shadow as I wanted to follow the Material Design guidelines. And as per your code is concerned I don't think there was a need for separate BlurBuilder. Also, the colored shadow is not visible as it should be like (Black) colored shadow, alpha will have different values for the different color shades.

@vipulasri vipulasri closed this Jan 14, 2018
@imaNNeo
Copy link
Contributor Author

imaNNeo commented Jan 14, 2018 via email

@vipulasri
Copy link
Owner

@imaNNeoFighT Thanks! I will look into it.

@vipulasri vipulasri reopened this Jan 14, 2018
@vipulasri vipulasri closed this Jan 14, 2018
@vipulasri
Copy link
Owner

@imaNNeoFighT can you raise the PR again with these changes which you proposed earlier ?

@imaNNeo
Copy link
Contributor Author

imaNNeo commented Sep 16, 2020

Yes let me check

@imaNNeo
Copy link
Contributor Author

imaNNeo commented Sep 16, 2020

I checked it, but it's a little hard for me (because the source branch is removed, and I don't have access to checkout to this pull request),
You can run this code to have these changes as a new branch in your environment,
then make a new PR for it.
git fetch origin merge-requests/{PR Id}/head:{yourBranchName}

For example for here:
git fetch origin merge-requests/2/head:shadow-color

@vipulasri
Copy link
Owner

@imaNNeoFighT thanks, actually I wanted you to contribute as you were the original contributor of this idea and code.
No issues, I will work on this.

@imaNNeo
Copy link
Contributor Author

imaNNeo commented Sep 17, 2020

I know, but I think I need to have access to this repo because I don't have access to the code.

@vipulasri
Copy link
Owner

@imaNNeoFighT Ohh I see. But as far as I know, You can also check the individual commits done and make the same changes again in the new Repo.

  1. Add BlurBuilder to separate blur function
  2. add shadowColor property
  3. fix shadowColor setter,getter bug and add shadowColor to updatable at…

@imaNNeo
Copy link
Contributor Author

imaNNeo commented Sep 17, 2020

Okay, I'm working on it.

@imaNNeo
Copy link
Contributor Author

imaNNeo commented Sep 17, 2020

Here you go, #32

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