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
Add: Reset button to global styles sidebar #25426
Add: Reset button to global styles sidebar #25426
Conversation
Size Change: +243 B (0%) Total Size: 1.17 MB
ℹ️ View Unchanged
|
packages/edit-site/src/components/editor/global-styles-provider.js
Outdated
Show resolved
Hide resolved
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.
Code-wise this is ready, although I'll defer to Noah for design input.
The only bit of feedback I have around IxD is:
- Can the reset button be disabled when there's no user changes (so it follows the same pattern than undo/redo)?
- "Reset all" is a bit confusing to me. Perhaps there's better alternatives along the lines of "Reset to defaults" or "Reset to theme"?
6ce2329
to
bb57785
Compare
bb57785
to
c00d25b
Compare
Hi @nosolosw, your feedback was addressed. @noahshrader in case there are some design improvements that should be done just let us know. |
This PR implements a reset all button on the global styles sidebar.
Fixes: #20868
How has this been tested?
I did some changes on the global styles sidebar and I verified that by pressing the global styles reset button I could reset all the changes.
Screenshots