ToggleGroupControl : Deprecate 36px default size

[vipul0425]

Part of #65751

What?

Deprecate the 36px default size in ToggleGroupControl.

Testing Instructions

• Unit tests pass
• Storybook stories should not log console warnings
• All code snippets in documentation (JSDoc, Storybook, README) should have the __next40pxDefaultSize prop enabled

[github-actions]

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: vipul0425 <[email protected]>
Co-authored-by: mirka <[email protected]>

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

[mirka]

Looking good!

Let's also add the __next40pxDefaultSize prop to the story for the ToolsPanel component.

I also pushed a proposal for a way to opt out from the warnings so certain components don't throw redundant warnings: 1b7eeec

[mirka]

Let's move this up to the Unreleased section.

[mirka]

I think we're also going to need a prop like this to prevent the redundant warnings when used inside another component: 1b7eeec

[vipul0425]

Hi @mirka should we also update the usage of ToggleGroupControl by adding this prop in other packages as well like block editor and library?

[mirka]

Not that I'm aware of. We only want to suppress the warning when ToggleGroupControl's size is dependent on a parent component's size, and the parent component is responsible for logging the warning. I'm not aware of any other cases like that.

[mirka]

Forgot to mention, the snapshot tests also need to be updated.

[mirka]

Thanks for the follow-ups! I fixed up a few minor things that were left, but we're good to go 🚀