-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[joy-ui][Input] Add debounce input demo #39300
Conversation
Netlify deploy previewBundle size report |
60fe218
to
9a85852
Compare
@sai6855 Hi, instead of showing an alert that disrupts the flow and also is not Joy UI design friendly, may be changed to something like the that: Input: Input |
Agree with @marcpachecog, the alert is a bit annoying. I think a simpler way is to have a text under the input that updates after the debounced period. |
@siriwatknp removed alert and added text below input |
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 @sai6855
preview: https://deploy-preview-39300--material-ui.netlify.app/joy-ui/react-input/#debounced-input
https://www.npmjs.com/package/react-debounce-input has 224k weekly downloads, so i think debounce is pouplar enough use-case that can go be a demo.