-
Notifications
You must be signed in to change notification settings - Fork 51
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
Slider text field #1492
base: main
Are you sure you want to change the base?
Slider text field #1492
Conversation
Seeing 2 x E2E failures in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks as always @cgjgh - another great addition, although, whilst it does bind well with the slider when changing from the slider side, two issues spotted when changing the value directly in the text input:
- It doesn't emit an event back into Node-RED when only changing the text field
- If you have steps in place, e.g. 0-10 with steps of 2, and you put in a number that isn't valid because the steps, but is valid because of the range - the slider will update correctly, but the text-field doesn't correct itself
I think you may need to bind the text-field to a different model value, then handle the events accordingly.
Description
Adds an optional text field for number input next to slider.
Screen.Recording.2024-11-21.214316.mp4
Related Issue(s)
Closes #1063
Checklist
flowforge.yml
?FlowFuse/helm
to update ConfigMap TemplateFlowFuse/CloudProject
to update values for Staging/ProductionLabels
area:migration
label