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

feat: added markers/ticks/labels and bg color for ranges #106

Merged
merged 3 commits into from
Mar 22, 2022

Conversation

sjdrew
Copy link

@sjdrew sjdrew commented Sep 14, 2021

Thanks for a great widget...

I have added additional input parameters to allow markers/ticks with optional labels, font specifications. As well added margin param (needed if labels involved) and bgOpacity property to the thresholds which if specified uses the threshold color with the applied opacity to form the background color for the range.

See below for results:

WaitWell_Admin

WaitWell_Admin

@ashish-chopra
Copy link
Owner

Thanks for contributing @stevedrew! This addition looks amazing. I will go through it soon!

@stanislav-kondriukov
Copy link

@ashish-chopra Hello! Will you merge this branch to master? I really need this functionality for my project.
Thank you in advance.

@ashish-chopra
Copy link
Owner

Yup, it's a long pending. I will try to do it over the weekend. Cheers!

@stanislav-kondriukov
Copy link

@ashish-chopra, Thank you very match! Cheers!

@stanislav-kondriukov
Copy link

@ashish-chopra, hello! Any status update here?

@maidadur
Copy link

@ashish-chopra hi, any updates on this issue?

@ashish-chopra ashish-chopra merged commit 68e8754 into ashish-chopra:master Mar 22, 2022
@maidadur
Copy link

@ashish-chopra hi, appreciate a lot for merging, can u also update the npm package? thanks!

@ashish-chopra
Copy link
Owner

v5.0.0 has been published. Thank you everyone for contributing and I apologize for the delay.

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.

5 participants