Feat: Moves size settings to inspector controls.

[vipul0425]

What?

Fixes: #65647

Why?

The Social Icons block has a 'size' setting that is placed in a toolbar dropdown menu labeled 'Size', but in other blocks it is there in inspector controls.

How?

This PR moves the size settings to inspector controls.

Testing Instructions

1. Add the Social Icons Block.
2. Size settings should be available in the inspector controls panel instead on block toolbar.

Screenshots or screencast

Screen.Recording.2024-09-30.at.1.21.49.PM.mov

[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: t-hamano <[email protected]>
Co-authored-by: hanneslsm <[email protected]>
Co-authored-by: richtabor <[email protected]>
Co-authored-by: afercia <[email protected]>

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

[t-hamano]

Thanks for the PR!

The ToggleGroupControl may look good in English locale, but not in other locales.

In particular, it will cause strange wrapping in CJK languages.

Japanese (ja)

繁體中文 (zh-TW)

The approach I can think of is to use actual numbers or icons instead of text, or use the SelectControl component. Also, note that as stated in this comment, this size variation may be expanded in the future.

[vipul0425]

Thanks @t-hamano for the review. If the number of options can increase in future then I think selectControl would be an ideal option provided we are not allowing any abstract value.

I have updated the PR with selectControl let me know if anything else needs to be updated 🙇

[t-hamano]

@vipul0425 Thanks for the update!

Discussion on this issue is still ongoing in #65647, and it also has been proposed to allow custom values. Let's move forward with this PR once the direction of the discussion is decided. Sorry for the confusion 🙇‍♂️