-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Duotone: Fix setup state for image block. #49949
Conversation
CC: @ajlende in case you worked on this. I wonder if there's a better way to enable duotone in the literal placeholder state 🤔 |
Size Change: +10 B (0%) Total Size: 1.37 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.
We've been reordering some of the CSS in the process of cleaning up some code. It may have been caused by that. Thanks for the fix!
Thank you! |
What?
You can apply duotone to the setup state of an image:
For decorative blocks like this and others (featured image, cover, media & text) this is a useful way to build wireframe patterns that communicate: "a duotone filtered image will go here". While it's not yet applied to all those blocks yet, it works for the image block.
But not the selected state:
Why?
This regressed in how filters were applied, presumably, but is easily fixed by increasing the specificity here.
Testing Instructions
Insert an image block. Apply duotone. Select the block. Duotone should not affect the buttons and text of the selected setup state.