-
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
Move Theme Editor to Tools when a block theme is active #36354
Comments
I created a Trac ticket for this 5 days ago, and it has two simple patches, one that moves the theme editor menu item, and one that moves both the theme editor and the plugin editor menu item, as suggested in ticket. I do not think that this should be solved by making changes to Gutenberg. |
I did suggest moving both editors as Carolina mentioned, but then I tried out the latest implementation and changed my mind. What I noticed is the Templates item is now the primary way of navigating between templates, and the Editor item is nothing more than a shortcut to the default template. I now think it would be clearer to remove the new Editor item altogether and just rely on Templates, allowing the old Theme Editor to stay where it is. |
Hmm yes that is worth considering in my opinion. |
That's not quite true. If you have a static home page set, then the Editor will load that template with the relevant page content in situ, enabling you to edit both at the same time. This lays the foundation for the broader "Site Editor" ambition where you're able to edit all aspects of your site at the same time. There are also issues like #36207 that will help make the Editor feel more like a holistic "Site editor" for 5.9. As we make these changes and add more features like browse mode, inline content creation, site setting management, etc, I suspect we'll see a divergence between the Site Editor and the Template Editor. The latter being the place you go to edit templates absent of any content. |
Thanks for the clarification. If content editing facilities are present, then wouldn't it be more appropriate for it to be elsewhere than Appearance? |
I'm not sure there is an appropriate section currently since this idea of cross-context editing doesn't really exist in WP just yet. When the experience is much richer (you're able to navigate to different content/templates, edit site settings, etc – hopefully for 6.0) we might consider elevating it to the top level. But for now editing style and template feels very theme-oriented, so the Appearance section probably makes the most sense. |
It appears you can say that again, because I notice that the Post Content block is not editable. I know it was editable in a previous version, so maybe this feature was removed, or maybe it's a problem on my end? |
I think it depends. Post Content in a query can't be edited, but Post Content in a page can be. |
I wasn't able to edit post content even outside of a query loop. |
Interesting, it's working fine for me with multiple themes. That may be worth a dedicated ticket if you have the time? |
Once the designs proposed in #29630 are implemented, we're going to have two fairly similar menu items under Appearance:
In many ways, the Site Editor will become a safer, more visual, user friendly version of the Theme Editor over time. There may still be a need for a code editor, but it will be a more advanced feature for power users.
Therefore it may make sense to de-emphasise the Theme Editor. This eases any confusion between "Editor" and "Theme Editor" in the Appearance menu, and puts an advanced further out of reach for less technical users.
One suggestion in 29630 was to move it to the Tools menu, which I think is a decent idea:
Any other thoughts / suggestions?
The text was updated successfully, but these errors were encountered: