-
Notifications
You must be signed in to change notification settings - Fork 3
Smart::NumberInput #579
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
Smart::NumberInput #579
Conversation
4db387f
to
ed03e50
Compare
You have run out of free Bugbot PR reviews for this billing cycle. This will reset on October 28. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
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.
lgtm overall just not sure where we stand on the loading state :)
loading: { | ||
description: '[OPTIONAL] Shows loading state with animated placeholder', | ||
table: { | ||
type: { | ||
summary: 'boolean' | ||
}, | ||
defaultValue: { summary: 'false' } | ||
}, | ||
control: { type: 'boolean' } | ||
}, | ||
placeholder: { | ||
description: '[OPTIONAL] Placeholder text shown during loading state', | ||
table: { | ||
type: { | ||
summary: 'string' | ||
}, | ||
defaultValue: { summary: 'undefined' } | ||
}, | ||
control: { type: 'text' } | ||
}, |
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.
I'm confused, does this actually work ? I thought you mentioned it was not possible in the PR description
What does this PR do?
Quick component for smart number input.
No loading state placeholder ( lack of space ) , the smart gradient is applied when loading state goes back to false, same pattern as other components
Related to: #
What are the observable changes?
Good PR checklist