-
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
Patterns: correctly color code unsynced patterns titles in site editor #52958
Conversation
…can be properly color coded
Size Change: +187 B (0%) Total Size: 1.44 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.
This is testing nicely for me!
✅ Templates display using synced color
✅ Synced patterns display using synced color
✅ Individual pages display using black as on trunk
✅ Unsynced patterns display using black
LGTM! ✨
packages/edit-site/src/components/header-edit-mode/document-actions/index.js
Outdated
Show resolved
Hide resolved
Thank you! This is working for me :) |
Co-authored-by: Andrew Serong <[email protected]>
What?
Makes sure the unsynced patterns do not have the purple title color in the site editor header bar.
Why?
Fixes: #52923
How?
Add an is-synced-entity class to document actions so synced entities can be properly identified and color coded.
Testing Instructions
In the site editor patterns section load a range of different patterns and template parts in the editor and check the color of the title and icon in the editor title bar are purple for synced patterns and black for unsynced patterns
Screenshots or screencast
Before:
After: