-
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
Widgets: Add Welcome Guide #31925
Widgets: Add Welcome Guide #31925
Conversation
1d2f9e1
to
b9ff22a
Compare
4276707
to
c8e3476
Compare
Size Change: +278 kB (+21%) 🚨 Total Size: 1.58 MB
ℹ️ View Unchanged
|
b9ff22a
to
aaa1f9f
Compare
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.
I've rebased to make the diff cleaner for review.
Works well and code looks good, nice work!
This really highlights how |
aaa1f9f
to
e3615f0
Compare
Why was this merged with failing tests? It's not because trunk is failing that that's ok? |
Testing. WordPress 5.7.2. Theme: Twenty Twenty. This is what I see: Welcome-Guide-Widget-screen.mp4Thank you for adding this feature! |
Description
Requires #31926.
Implements part of #30887.
Adds
WelcomeGuide
to the widgets screen. This is mostly copied fromWelcomeGuide
inedit-post
.We show the "new user" copy when all widgets are block widgets, because when this feature is merged to Core, we will update
wp_install_defaults
(here) to set up a sidebar containing block widgets in new WordPress installations. We can't do this in the Gutenberg plugin, though.How has this been tested?
localStorage.clear()
.Screenshots
Checklist:
*.native.js
files for terms that need renaming or removal).