Search Block: Master PR

[AmandaRiu]

Related to #3073. Instead of creating a new PR for every PR reviewed in Gutenberg, Cameron and I thought it'd make more sense to just have a master PR for updating the gutenberg reference for testing but not merging until the Search block has been added to the initial-html.js or has breaking tests. This way this PR can act as a record of all the PRs and it will be easier to parse.

• The last entry in the table below with the ☑️ will be the active gutenberg reference under test.
• Once the gutenberg PR has been merged it will be updated with the ✅ symbol.

This PR is now ready for merge once the last PR in the table below has been approved and merged

Gutenberg PR	Merged
Search Block - Add to inserter	✅
Search Block: Add block toolbar options	✅
Search Block: Basic UI	✅
Search Block: Add button position UI support	✅
Search Block: Width Setting	✅
Search Block: Fix iOS Focus Issues	✅
Search Block: Move Block Toolbar options to settings panel	✅
Search block: Styling	✅
Search block: Accessibility	✅
Search block: Unit Tests	✅
Search block: Long button text handling	✅
Search block: UI Tests	✅

To test

Click the active Gutenberg PR in the table to view full test instructions.

PR submission checklist:

• I have considered adding unit tests where possible.
• I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

[peril-wordpress-mobile]

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

[guarani]

Being new on the block 😏, I think the search block gets an invite to the "other block list", as suggested here.
I think it's easy to miss so thought I'd mention it in case it's not on your radar.

[dcalhoun]

@AmandaRiu I attempted to test this branch within WPiOS via setting the gutenberg ref to this branch's last commit. I also attempted by checking out this branch, npm run bundle, and LOCAL_GUTENBERG=true bundle exec pod install. However, the Search block would not show up within the block inserter menu. Am I overlooking something?

[AmandaRiu]

Oh thank you @guarani for this information:

Being new on the block 😏, I think the search block gets an invite to the "other block list", as suggested here.
I think it's easy to miss so thought I'd mention it in case it's not on your radar.

I had completely missed that.

[AmandaRiu]

@AmandaRiu I attempted to test this branch within WPiOS via setting the gutenberg ref to this branch's last commit. I also attempted by checking out this branch, npm run bundle, and LOCAL_GUTENBERG=true bundle exec pod install. However, the Search block would not show up within the block inserter menu. Am I overlooking something?

@dcalhoun I think it may have been because I hadn't added the search block to the supported blocks in gutenberg mobile as @guarani had pointed out in this comment.. I've added it now though so it should work 🤞

[dcalhoun]

LGTM. I tested this, with a focus on WordPress/gutenberg#30783, on an iPhone SE and Samsung Galaxy S20.

We'll just need to change the gutenberg ref to a merge commit before merging this work.