-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
[addon-knob] Display for Number value, min, max when slider is used. #2445
Comments
Looks like it may be relatively easy to implement using Feel free to send a PR, but please base it on |
Unfortunately, even if it's within the spec, no browsers support it fully right now (codepen https://codepen.io/cilice/pen/ypBWKM)
Unfortunately, latest Safari supports neither hash marks nor labels just like Firefox. |
Displaying the currentValue would be nice. Especially while development when you're tweaking some default values. |
Hi everyone! Seems like there hasn't been much going on in this issue lately. If there are still questions, comments, or bugs, please feel free to continue the discussion. Unfortunately, we don't have time to get to every issue. We are always open to contributions so please send us a pull request if you would like to help. Inactive issues will be closed after 60 days. Thanks! |
I am very interested in the feature request. |
@mkochendorfer want to open a PR? |
I don't have the time this hit this right now, but possibly in the coming months. |
I can work on this in the coming days. I'd propose something simple to get started: Minimum on the left, and "Current / Maximum" on the right. The UI can be enhanced later with tick marks and so on if browsers start supporting that more. |
#2445 Add min, value, and max labels to range knob
Released as |
@Hypnosphi any chance of (#3128) being part of v3.4.0 release? |
Sorry, 3.4 is feature-frozen |
figured. Thought it was worth asking :) 👍 |
Feature Request:
It would be nice to have Value, Min and Max displayed when using the number knob range to give the user some context before they grab the slider.
The text was updated successfully, but these errors were encountered: