-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Restore the browse all button in the site editor #47054
Conversation
Size Change: -9 B (0%) Total Size: 1.33 MB
ℹ️ View Unchanged
|
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.
Thanks Riad! Noting though that the connected issue is different and it just mentioned the missing Browse all
button.
Oh right, I'll leave the issue open then :) |
Flaky tests detected in c0b65d7. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/3891993463
|
Thanks for bringing it back @youknowriad 🙇🏻 |
Looks like it's gone again. Was this intentional? Created an issue: #52526 |
What?
In #44770 the "browse all" button in the inserter has been inadvertently removed from the site editor because the "inserter" state has been moved to the site editor store but the callback was not being provided any more. This PR fixes that by passing the callback properly.
Testing Instructions
1- open the site editor
2- trigger the in-between inserter
3- the "browse all" button should be visible and working properly.