-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[LoadingButton] Introduce new component #21389
Conversation
@material-ui/lab: parsed: +4.18% , gzip: +3.11% Details of bundle changes.Comparing: c4351c6...98b6c6c Details of page changes
|
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.
See #21389 (comment)
Would |
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.
Much better than my first iteration
Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com>
Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com>
Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com>
Co-authored-by: Olivier Tassinari <olivier.tassinari@gmail.com>
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.
This should go in the lab first.
@eps1lon that makes sense. Should we leave the examples on the Buttons page, or create a new page under lab for the Busy buttons? |
@mnajdova "on the same page" is a viable option, for instance: |
Ok, will leave the examples then as they were. |
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.
Great work!
When will this be released? Or is it already released? If so, in which version? |
It was released in v5.0.0-alpha.1 |
just curious, will it be a new component in a library or it will be replacement for the |
New component |
Inspired by #17810
Closes #7223
Some considerations:
pendingIndicatorPosition
for allowing customers to specify where thependingIndicator
will be shown: 'start' | 'center' | 'end'. It is replacing respectively thestartIcon
,children
or theendIcon
part.pendingIndicator
isCircularProgress
Button
to disabled if it is in pending state, although this means that the contained buttons will end up with the grey disabled background.. Not sure if this is expected, but I am open for other ideaspendingIndicator
is absolutely positioned inside theButton
, depending on thependingIndicatorPosition
prop, in order to avoid width changes of theButton
Benchmark
The implementation was benchmarked with the following sources. I have noticed that the loading and disabled states are often styled independently.
Evergreen
Blueprint
Vuetify
Cosmos
Antd
Element
Sancho
SemanticUI
Quasar
Base UI
Atlaskit