[wp-trac] [WordPress Trac] #38375: Flatten/rename TwentySeventeen directory structure
WordPress Trac
noreply at wordpress.org
Thu Oct 20 05:10:53 UTC 2016
#38375: Flatten/rename TwentySeventeen directory structure
---------------------------+-------------------------
Reporter: dd32 | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.7
Component: Bundled Theme | Version:
Severity: normal | Keywords: 2nd-opinion
Focuses: |
---------------------------+-------------------------
With TwentySeventeen merged in [38833] it's a little more obvious about
the directory structure than presented on Github.
Burying `css/`, `images/`, and `js/` within an `assets/` folder isn't
particularly useful and isn't immediately obvious as to the contents IMHO,
promoting these to a root level entry can't hurt.
Likewise as @bronsonquick mentioned to me, having template parts (used
with `get_template_part()`) within a `components/` folder (where no
SCSS/JS/etc exists as 'components') feels weird and confusing to me as a
'newbie theme developer'.
My immediate thought is that `/components/*/*.php` would feel better as
`template-parts/*.php`.
I don't feel super strongly on these, but it might make it a little more
accessible to those who have not worked on complex themes structured in
this format.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/38375>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list