Components: Remove unnecessary Tooltip component from linked button

[im3dabasia]

Fixes: #66271

What?

Quick code quality PR to remove unnecessary usage of the Tooltip component.

Why?

Improve on code quality

How?

Remove Tooltip component and leverage the built-in Button label prop.

Testing Instructions

border-box-control-linked button

1. Select a Group block.
2. In the block Settings panel, go to Styles > Border & Shadow
3. Hover or focus the 'Unlink sides / Link sides' button.
4. Observe the button tooltip is unchanged: 'Unlink sides' or 'Link sides' depending on the UI state.
5. Observe the accessible name (aria-label) of the Button is unchanged and matches the tooltip.

spacing-sizes-control linked button

1. Select a Spacer block.
2. In the block Settings panel, go to Styles > Dimensions > Margin
3. Hover or focus the 'Unlink sides / Link sides' button.
4. Observe the button tooltip is unchanged: 'Unlink sides' or 'Link sides' depending on the UI state.
5. Observe the accessible name (aria-label) of the Button is unchanged and matches the tooltip.

Screenshots

border-box-control-linked button

spacing-sizes-control linked button

box-control-linked-button (I tested in the in the boxcontrol story)

[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: im3dabasia <[email protected]>
Co-authored-by: ciampo <[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.

[im3dabasia]

Hi @afercia,
When you have a moment, could you please review my PR?

[afercia]

Can we try to remove this View component and set the className directly on the Button? Not sure the View is necessary at all (it's just a div element that wraps the button).

[afercia]

@im3dabasia thanks for working on this.
I left a note about removing an apparently unnecessary View component.

Also, a changelog entry should be added to the 'Unreleased' section of the packages/components/CHANGELOG.md file referencing the changes in the BorderBoxControl and BoxControl components. For an example, see this entry that references two components.