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.

Sign up for GitHub

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

Jump to bottom

activeColorOpacity added. #81

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

activeColorOpacity added. #81

wants to merge 1 commit into from

Conversation

oguzhanaksoy
Copy link

activeColorOpacity property added and used.

@oguzhanaksoy
Copy link
Author

oguzhanaksoy commented Apr 22, 2021 •
edited
Loading

I am building an application and I have 2 different colors which are Light Blue and Orange.
I would like to have this feature on the package.
So I made this changes.

Here how it looks like with 0.2 opacity:
Sample

Note: this is my first PR on the Github environment. Please don't harsh me if I did something wrong :)

@pedromassango pedromassango self-requested a review April 23, 2021 15:37
@pedromassango
Copy link
Owner

Hi, thanks for this contribution,

I will review this asap.

pedromassango
pedromassango requested changes Jun 22, 2021
View reviewed changes
Copy link
Owner

@pedromassango pedromassango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please add some tests for this? Tests will make sure this does not break in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pedromassango pedromassango pedromassango requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@oguzhanaksoy @pedromassango