Skip to content
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

website: ButtonGroup #1162

Merged
merged 11 commits into from
Mar 31, 2023
Merged

website: ButtonGroup #1162

merged 11 commits into from
Mar 31, 2023

Conversation

gretanausedaite
Copy link
Contributor

@gretanausedaite gretanausedaite commented Mar 28, 2023

Changes

Updated ButtonGroup documentation page
Solves ButtonGroup part of #961

Testing

N/A

Docs

N/A

@gretanausedaite gretanausedaite requested a review from a team as a code owner March 28, 2023 14:06
@gretanausedaite gretanausedaite requested review from a team, FlyersPh9 and adamhock and removed request for a team March 28, 2023 14:06
@gretanausedaite gretanausedaite self-assigned this Mar 28, 2023
@FlyersPh9 FlyersPh9 added the documentation Improvements or additions to documentation label Mar 28, 2023
Copy link
Contributor

@adamhock adamhock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The examples and formatting of information all look good. I've added a few proofreading comments, but other than that it looks good to me!

apps/website/src/pages/docs/buttongroup.mdx Outdated Show resolved Hide resolved
apps/website/src/pages/docs/buttongroup.mdx Outdated Show resolved Hide resolved
apps/website/src/pages/docs/buttongroup.mdx Outdated Show resolved Hide resolved
apps/website/src/pages/docs/buttongroup.mdx Outdated Show resolved Hide resolved
apps/website/src/pages/docs/buttongroup.mdx Outdated Show resolved Hide resolved
apps/website/src/pages/docs/buttongroup.mdx Outdated Show resolved Hide resolved

### Overflow

If there is not enough space to display all of the buttons, the buttons can truncate into an overflow menu.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does it make sense to link the word menu to the menu component?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a link to dropdown menu button and updated overflow sample to be more simple

@gretanausedaite gretanausedaite added this pull request to the merge queue Mar 31, 2023
Merged via the queue into main with commit 1feac5f Mar 31, 2023
@gretanausedaite gretanausedaite deleted the greta/button-group branch March 31, 2023 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants