-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Docs/update button component props #24013
Docs/update button component props #24013
Conversation
Elaborates more on ToolbarComponent information, mentioning its accessibility features. Co-authored-by: Daniel Richards <daniel.p.richards@gmail.com>
Proper documentation links on ToolbarButton page. Co-authored-by: Marcus Kazmierczak <marcus@mkaz.com>
Proper links usage on ToolbarButton documentation page. Co-authored-by: Marcus Kazmierczak <marcus@mkaz.com>
Proper links usage on ToolbarButton documentation page. Co-authored-by: Marcus Kazmierczak <marcus@mkaz.com>
Proper links usage on ToolbarButton documentation page. Co-authored-by: Marcus Kazmierczak <marcus@mkaz.com>
…ttonGroup documentation.
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.
Seems like a nice addition, I don't understand why the formatting change is needed though.
- Toolbars | ||
- Modals | ||
- Forms | ||
- Toolbars |
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.
Why all lists are not formatted differently?
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.
Hi @youknowriad. I am not sure why, but after installing the prettier extension as in here, every time I saved my file the one space that I inserted was converted to two spaces ¬¬'. Is there anything that I should change in my pretter settings?
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 think it's probably fine because we don't have strong linting checks for markdown. 👍
Description
Added missing props to the Button component props list, such as
className
,target
,icon
,showTooltip
and others related. Also, shows the props list more alike how it is done in other components: a list instead of a table.Types of changes
Documentation improvements