Skip to content

Conversation

@im3dabasia
Copy link
Contributor

@im3dabasia im3dabasia commented Jul 29, 2025

Closes: #70949

What?

Omitted label prop from the dropdownMenuProps used inside ToolsPanelHeaderProps

@im3dabasia im3dabasia changed the title [WIP]: Handle label fallback for dropdownMenu Handle label fallback for dropdownMenu Jul 30, 2025
@im3dabasia im3dabasia marked this pull request as ready for review July 30, 2025 05:57
@im3dabasia im3dabasia requested a review from ajitbohra as a code owner July 30, 2025 05:57
@github-actions
Copy link

github-actions bot commented Jul 30, 2025

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: im3dabasia <im3dabasia1@git.wordpress.org>
Co-authored-by: mirka <0mirka00@git.wordpress.org>
Co-authored-by: t-hamano <wildworks@git.wordpress.org>

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

@t-hamano t-hamano added [Type] Code Quality Issues or PRs that relate to code quality [Package] Components /packages/components labels Jul 31, 2025
@t-hamano
Copy link
Contributor

@im3dabasia Thanks for the PR!

Can you update this PR according to this comment?

@t-hamano t-hamano requested a review from a team July 31, 2025 01:01
Comment on lines -80 to +83
dropdownMenuProps?: React.ComponentProps< typeof DropdownMenu >;
dropdownMenuProps?: Omit<
React.ComponentProps< typeof DropdownMenu >,
'label'
>;
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this format is correct, but I don't know how to describe it in the README 😅

cc @WordPress/gutenberg-components

Additionally, I think we need to update here as well:

dropdownMenuProps?: React.ComponentProps< typeof DropdownMenu >;

Copy link
Member

Choose a reason for hiding this comment

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

I don't know how to describe it in the README 😅

Mm, I'm fine with this for now, it's not crucial information. And we won't have to think about this once these are converted to an autogenerated readme!

Copy link
Member

@mirka mirka left a comment

Choose a reason for hiding this comment

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

I guess this is good to go? Sorry for the delay!

@mirka mirka merged commit b754e9f into WordPress:trunk Nov 2, 2025
41 of 43 checks passed
@github-actions github-actions bot added this to the Gutenberg 22.1 milestone Nov 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Package] Components /packages/components [Type] Code Quality Issues or PRs that relate to code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ToolsPanel: dropdownMenuProps type requires unnecessary label property

3 participants