Gallery block: Refactor "Settings" panel to use Toolspanel

[Mayank-Tripathi32]

part of #67813
resolves #67893

Before

After

[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: Mayank-Tripathi32 <[email protected]>
Co-authored-by: fabiankaegy <[email protected]>

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

[Mayank-Tripathi32]

Working on fixing unit test

[fabiankaegy]

@Mayank-Tripathi32 looks like there is already a PR for this one #67904

But that one also has the same issue with the unit tests. So feel free to collaborate in case you already have a solution for the tests :)

[Mayank-Tripathi32]

@Mayank-Tripathi32 looks like there is already a PR for this one #67904

But that one also has the same issue with the unit tests. So feel free to collaborate in case you already have a solution for the tests :)

I tried to figure it out, but most issues occur on mobile, and I'm unsure how to fix them. If you have any resources, I'd be happy to try again.

[t-hamano]

Thanks for the PR!

I think the unit test failure is based on the fact that ToolsPanel and ToolsPanelItem are not available in the React Native version. There are two solutions:

1. Split the Edit component into a web version and a React native version. That is, create a edit.native.js file for the React Native version. Leave the Edit component unchanged in the React native version.
2. Render the component conditionally using Platform.isWeb or Platform.isNative.

I think the latter approach is better for the Gallery block, since the Platform is already used for block controls etc.

[Mayank-Tripathi32]

i think we can move with #67904 as it seems to have unit tests fixed. Will close this.