emulsify-ds / emulsify-wordpress-theme

WordPress theme built with Storybook and Webpack
MIT License
24 stars 10 forks source link

Sub-theming in WordPress? #3

Closed evanmwillhite closed 2 years ago

evanmwillhite commented 4 years ago

I don't think we're "subtheming" correctly (or at all), so this is a task to make sure we are either doing it correctly or to track the work to fix it. It may entail the following, but honestly I'm not sure:

Helpful? links:

mirzu commented 4 years ago

Anyone with Wordpress Experience could take a look at this.

terrance-orletsky-d7 commented 3 years ago

I may be interested in looking at this, if the intended result will suit the needs of the project I'm working on.

@evanmwillhite Can you please elaborate on what the Acceptance Criteria would be for the solution you propose, and what the key blockers/areas of non compliance are, presently?

I acknowledge you have have created a list as a 'starting point' in the description, but I'm wondering if you (or anyone else?) has looked at this further since posting, so that I am not duplicating effort.

evanmwillhite commented 3 years ago

@terrance-orletsky-d7 that is awesome! Yes, we have actually nailed down a path forward with this but I hadn't recorded anything in here yet so thanks for the reminder. Here's our list as it stands currently below:

Specific tasks

  1. Move Timber starter code from functions.php into a new inc/timber.php file. Where the new file gets included in the functions.php file, add some helpful comments as to what this functionality is.
  2. Move Emulsify-specific Twig functions in functions.php to a new inc/emulsify.php file. Again, where the new file gets included in the functions.php file, add some helpful comments as to what this functionality is.

General Tasks

  1. Increase project documentation both within project README and https://docs.emulsify.info/
  2. Run theme sniffer on project
  3. Look over popular Wordpress themes (e.g., twentytwentyone) and make sure that we follow conventions where it makes sense. We may break conventions in some ways, but let's make sure they're purposeful. One thing we could do for example is to follow the template directory structure for Wordpress-specific template files.
mikeethedude commented 2 years ago

A lot has changed with how we approach WP themes since this was originally written. I'm going to close this issue in favor of a new one around making this theme cleaner and closer to best practices.