Skip to content
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

Open
jameskoster opened this issue Nov 9, 2021 · 10 comments
Open

Move Theme Editor to Tools when a block theme is active #36354

jameskoster opened this issue Nov 9, 2021 · 10 comments

Comments

Projects
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@jameskoster
Copy link
Contributor

@jameskoster jameskoster commented Nov 9, 2021

Once the designs proposed in #29630 are implemented, we're going to have two fairly similar menu items under Appearance:

  • Editor (beta) – This is where you go to visually customise your home page and styles in the "Site Editor".
  • Theme Editor – This is a code editor for theme files

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:

Screenshot 2021-11-09 at 16 02 10

Any other thoughts / suggestions?

@jameskoster jameskoster added this to 📥 To do in WordPress 5.9 Must-Haves via automation Nov 9, 2021
@carolinan
Copy link
Contributor

@carolinan carolinan commented Nov 10, 2021

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 also do not think it is a good idea to have them in two places depending on what type of theme is active.
I believe that always having the menu items in the same location makes them easier to use.

Loading

@carlomanf
Copy link

@carlomanf carlomanf commented Nov 10, 2021

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.

Loading

@carolinan
Copy link
Contributor

@carolinan carolinan commented Nov 10, 2021

Hmm yes that is worth considering in my opinion.

Loading

@jameskoster
Copy link
Contributor Author

@jameskoster jameskoster commented Nov 10, 2021

Editor item is nothing more than a shortcut to the default template

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.

Loading

@carlomanf
Copy link

@carlomanf carlomanf commented Nov 10, 2021

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.

Thanks for the clarification. If content editing facilities are present, then wouldn't it be more appropriate for it to be elsewhere than Appearance?

Loading

@jameskoster
Copy link
Contributor Author

@jameskoster jameskoster commented Nov 11, 2021

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.

Loading

@carlomanf
Copy link

@carlomanf carlomanf commented Nov 13, 2021

this idea of cross-context editing doesn't really exist in WP just yet.

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?

Loading

@jameskoster
Copy link
Contributor Author

@jameskoster jameskoster commented Nov 15, 2021

I think it depends. Post Content in a query can't be edited, but Post Content in a page can be.

Loading

@carlomanf
Copy link

@carlomanf carlomanf commented Nov 15, 2021

I wasn't able to edit post content even outside of a query loop.

Loading

@jameskoster
Copy link
Contributor Author

@jameskoster jameskoster commented Nov 16, 2021

Interesting, it's working fine for me with multiple themes. That may be worth a dedicated ticket if you have the time?

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment