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

ToolbarButton: Fix docs for accessibleWhenDisabled #63140

Merged
merged 1 commit into from
Jul 5, 2024
Merged

Conversation

mirka
Copy link
Member

@mirka mirka commented Jul 4, 2024

What?

Fix the Storybook props table for ToolbarButton so it shows the correct default for the accessibleWhenDisabled prop.

Why?

The __experimentalIsFocusable prop was stabilized and renamed to accessibleWhenDisabled (#62282) in the Button component where this component inherits its props from. The default for the prop in Button is false, whereas here it is functionally equivalent to true.

Testing Instructions

The Storybook props table for the ToolbarButton component (subcomponent of Toolbar) should show the correct default for accessibleWhenDisabled.

Screenshots or screencast

Storybook props table showing the correct default

@mirka mirka added [Type] Developer Documentation Documentation for developers [Package] Components /packages/components Storybook Storybook and its stories for components labels Jul 4, 2024
@mirka mirka requested a review from a team July 4, 2024 17:12
@mirka mirka self-assigned this Jul 4, 2024
@mirka mirka requested a review from ajitbohra as a code owner July 4, 2024 17:12
Copy link
Contributor

@DaniGuardiola DaniGuardiola left a comment

Choose a reason for hiding this comment

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

good stuff

Copy link

github-actions bot commented Jul 4, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: mirka <[email protected]>
Co-authored-by: DaniGuardiola <[email protected]>
Co-authored-by: tyxla <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Copy link
Member

@tyxla tyxla left a comment

Choose a reason for hiding this comment

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

👍

@mirka mirka merged commit 88f44fd into trunk Jul 5, 2024
70 of 71 checks passed
@mirka mirka deleted the toolbar-button-docs branch July 5, 2024 12:07
@github-actions github-actions bot added this to the Gutenberg 18.8 milestone Jul 5, 2024
carstingaxion pushed a commit to carstingaxion/gutenberg that referenced this pull request Jul 18, 2024
Co-authored-by: mirka <[email protected]>
Co-authored-by: DaniGuardiola <[email protected]>
Co-authored-by: tyxla <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Components /packages/components Storybook Storybook and its stories for components [Type] Developer Documentation Documentation for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants