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

Discussion: What should take over for the Theme Editor? #30695

Closed
paaljoachim opened this issue Apr 9, 2021 · 6 comments
Closed

Discussion: What should take over for the Theme Editor? #30695

paaljoachim opened this issue Apr 9, 2021 · 6 comments
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Type] Discussion For issues that are high-level and not yet ready to implement. [Type] Enhancement A suggestion for improvement.

Comments

@paaljoachim
Copy link
Contributor

paaljoachim commented Apr 9, 2021

In this issue:
#29630
@jameskoster is exploring new drop down links within the Appearance menu that is related to Full Site Editing.

Appearance-themes-screen

One of the links that is being removed is the Theme Editor the area where one can go directly to the various files to modify example the functions.php file to add in code snippets etc, style.css and other files within the themes area.

What should take over so that we can easily go in and make adjustments directly in the files?

@paaljoachim paaljoachim added [Feature] Full Site Editing [Type] Discussion For issues that are high-level and not yet ready to implement. [Type] Enhancement A suggestion for improvement. labels Apr 9, 2021
@paaljoachim paaljoachim changed the title Discussion: What show take over for the Theme Editor? Discussion: What should take over for the Theme Editor? Apr 10, 2021
@celloexpressions
Copy link

Is there a reason that the theme code editor wouldn't work with the new FSE features? The Additional CSS feature in the customizer was envisioned as a potential long-term replacement for the theme code editor. If there is consensus around deprecating the theme file editor, that could be a potential replacement. See the original feature proposal for more information.

@carolinan
Copy link
Contributor

carolinan commented Apr 12, 2021

This menu item is still needed and should not be removed.
Adding new CSS only is not the equivalent to editing theme files.

There are theme files that are not related to templates or CSS like a functions.php file that is needed to include other functional files like:
register block patterns
register block styles, enqueue CSS for the block styles -Block style CSS can not be created using theme.json because of the wp-custom naming convention
enqueue JS

There will also be a period where this is needed until global styles are completed -for example to enqueue remote fonts -
and until there is a code editor mode in the site editor.

@jameskoster
Copy link
Contributor

Since Templates, Styles, Header, Footer etc can each be considered aspects of theme editing, might it be worth changing the label on this menu item? I worry that the "Theme Editor" link might inadvertently lure newer users to an unnecessarily advanced (and potentially dangerous) UI rather than the newer (user-friendlier) alternatives.

Perhaps something like Appearance > Advanced, or Appearance > Code Editor would better describe how this feature sits along side the other site editing functions?

@bkjproductions
Copy link

bkjproductions commented Apr 16, 2021

The existing code/theme editor is one of those things that usually I have to disable for clients anyway. I would wonder why have it at all, now that the block editor seems to require more serious programming skills (if you're going to mess around in PHP/JS/React) and allowing the novice to willy-nilly change code might encourage sloppy coding.

@annezazu annezazu added [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") and removed [Feature] Full Site Editing labels Jul 24, 2023
@annezazu
Copy link
Contributor

Noting that it's now called the Theme File Editor and, for block themes, we simply show "Themes" and "Editor".

Screenshot 2023-10-27 at 8 37 34 AM

I think this issue can be closed out as I'm not sure how relevant it is today. Happy to reopen of course if I'm missing something or if we can get some fresh thoughts :)

@paaljoachim
Copy link
Contributor Author

Wow nice! Thank you for sharing Anne! I have not thought about this aspect of WordPress for quite a while.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Type] Discussion For issues that are high-level and not yet ready to implement. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

6 participants