-
Notifications
You must be signed in to change notification settings - Fork 58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable Inserter Block Search #3735
Conversation
Add Inserter Block Search note
Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved via WordPress/gutenberg#33559 (review), I left one nitpick here but not a blocker.
@@ -1,5 +1,6 @@ | |||
Unreleased | |||
------ | |||
* [***] Add Inserter Block Search [https://github.com/wordpress-mobile/gutenberg-mobile/issues/2570] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick: It might be beneficial to editorial to have a more descriptive (longer, more explicit) release note that they can shorten down. The link doesn't provide much detail on how this feature looked in the end, so I don't think they'll find much additional information there.
Given the feedback regarding the Android bottom sheet animation I'm nocking this one back to the next release. |
Closing in favor of #3867 |
Companion PR WordPress/gutenberg#33559
To test:
Follow test steps here WordPress/gutenberg#33237
PR submission checklist: