-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Global Styles: Add Welcome Guide toggle #37180
Conversation
Size Change: -9 B (0%) Total Size: 1.11 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 for the PR George!
I think some changes will be needed here as well: https://github.com/WordPress/gutenberg/blob/trunk/packages/edit-site/src/components/welcome-guide/index.js#L14, no ? 🤔
You can see in my recording when we select the global styles
and then click welcome guide
once (store update for feature) the guide is not rendered but if we click back to settings
it's rendered then.
Screen.Recording.2021-12-07.at.2.49.22.PM.mov
Great catch, Nik. The issue should be fixed now. |
6c28d76
to
f7780a2
Compare
@noisysocks, this got recently punted from the 5.9 Must-Haves list. Should we add it back? |
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.
Looks good, thanks George! 💯
I punted everything with the enhancement label. Looks small though, let's add the backport label. |
* Global Styles: Add Welcome Guide toggle * Check isActive state locally in components
Description
PR adds the Welcome Guide toggle button to the "More Global Styles Actions" dropdown.
Resolves #36544
How has this been tested?
Screenshots
Types of changes
Enhancement
Checklist:
*.native.js
files for terms that need renaming or removal).