-
Notifications
You must be signed in to change notification settings - Fork 37
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
added label to TransferListMain example. #1541
added label to TransferListMain example. #1541
Conversation
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.
There is no difference between the TransferList.main and TransferList.withlabel. Also, if the label is required, then "With Label" is no longer a variant; it is the default.
@mayank99, was there a reason why these two nearly identical examples existed? Were we planning to document more variants in the future? If there's no special reason, let's remove this TransferList.withLabel along with the "Variants" headings.
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.
#1541 (review) can be tackled in another PR
Agreed, we can remove the "withlabel" example, and document that labels should always be provided. |
Changes
Added area-label to TransferListMain example. Fixed a11y violation.
Testing
Before:
After:
Docs
N/A