-
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
Output post classes in the editor #60642
Conversation
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
👋 Thanks for your first Pull Request and for helping build the future of Gutenberg and WordPress, @huubl! In case you missed it, we'd love to have you join us in our Slack community. If you want to learn more about WordPress development in general, check out the Core Handbook full of helpful information. |
39d01bf
to
d8e437c
Compare
My main flag here is that That being said, I don't think that's a blocker since this is a well known property of the REST API already. |
Thanks for making those changes! This looks good to me from a REST API perspective. |
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 @huubl! I've left a couple of small comment where we can simplify and it should be good to land.
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.
Thank you! Let's 🚢
What happens when you change categories etc.? These things don't remain constant in the editor like they do on the front-end. |
What do you mean? Classes are applied from REST API using |
If you edit a post and change the categories, the generated classes for these categories should change too? Same for status, featured image and a lot of these properties. |
Ah, if this is just for the query block and not for editable content in a template, then I guess it's not relevant. At some point we'll have to apply this class outside the query block as well and make it dynamic. |
The query block can be dynamic as well, it can show posts that are "dirty". So this is more of an escape hatch I would say. |
@youknowriad what do you mean with this? |
@ntsekouras I mean posts with unsaved changes. |
@anton-vlasenko or @TimothyBJacobs Would it be possible to make a backport PR for this? |
See WordPress/gutenberg#60642. See #6716. Fixes #61360. Props antonvlasenko, timothyblynjacobs, ellatrix, oandregal. git-svn-id: https://develop.svn.wordpress.org/trunk@58326 602fd350-edb4-49c9-b593-d223f7449a82
See WordPress/gutenberg#60642. See WordPress/wordpress-develop#6716. Fixes #61360. Props antonvlasenko, timothyblynjacobs, ellatrix, oandregal. Built from https://develop.svn.wordpress.org/trunk@58326 git-svn-id: http://core.svn.wordpress.org/trunk@57783 1a063a9b-81f0-0310-95a4-ce76da25c4cd
See WordPress/gutenberg#60642. See WordPress/wordpress-develop#6716. Fixes #61360. Props antonvlasenko, timothyblynjacobs, ellatrix, oandregal. Built from https://develop.svn.wordpress.org/trunk@58326 git-svn-id: https://core.svn.wordpress.org/trunk@57783 1a063a9b-81f0-0310-95a4-ce76da25c4cd
What?
This pull request addresses issue #33887 (and I think #10640 is related to this) by introducing post classes within the query loop in the Gutenberg editor.
Note:
I would like to know if the approach in this pull request is in the right redirection.
Why?
This allows users to target specific elements and apply custom styles to them.
How?
Extends the WordPress REST API to include post classes in the response data for all public post types.
Alternatives
Alternative using a new REST API route to retrieve CSS classes of a specific post: #60645
Recreating all the classes in JS along with a JS filter could be an alternative solution.
Testing Instructions
Insert a Query block and observe the post classes added next to the
wp-post-block
class in the block inspector.Testing Instructions for Keyboard
--
Screenshots or screencast