<div dir="ltr"><div>The reason I say it's not something we should be concerned with is because, for the moment, our focus needs to simply be on making sure themes actually work correctly.  With all the work we've done over the years, I still don't feel like we're at a point where we're getting the kind of quality code that we should be getting from theme authors.  As I look through themes today, even after they've been reviewed and updated and reviewed and updated, there's still quite a bit of code that really should be better.</div><div><br></div><div>I consider recommending not to use `@import` right there in line with the Theme Unit Tests.  It's great if theme authors do it, but we've got too much on our plates as reviewers right now.</div><div><br></div><div>We still have problems with basic stuff like getting people to license their themes correctly, using the right template tags, and learning the difference between "home" and "front page".  To top all of that off, themes are only getting more complicated.</div><div><br></div><div>I think discussions on the performance of doing something one way vs. another way are great, but I don't think we're there yet in terms of making this sort of thing a requirement under the guidelines.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 3, 2014 at 5:35 PM, Tammie Lister <span dir="ltr"><<a href="mailto:karmatosed@gmail.com" target="_blank">karmatosed@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Whilst I'm not advocating we do it right now, it's food for thought.<div><br></div><div>Personally, I don't think it's far reaching. Not when we want to encourage best practices and that is one. It a little depends on context, which a guideline would outline.</div><div><br></div><div>Perhaps we're a little early, perhaps it's just a good thing we are having this discussion as it puts out there the fact that import is not the right path. I personally, would say never but I know that there are with anything a range of opinions on that. </div><div><br></div><div>Outside of WPTRT it's always my recommendation you avoid imports in php/html files. The exception to that rule I'd suggest would be Sass or other pre-processing inclusions. Again, my own opinion.</div><div><br></div><div>Regards</div><span class="HOEnZb"><font color="#888888"><div>Tammie Lister</div><div><div><br></div><div><br></div></div></font></span></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Fri, Oct 3, 2014 at 11:00 PM, Justin Tadlock <span dir="ltr"><<a href="mailto:justin@justintadlock.com" target="_blank">justin@justintadlock.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr"><div>Stopping the use of @import is probably a little far-reaching to be honest.  I don't think it's something we need to be actively looking for.  At best, I think we should go as far a making it "recommended" to not use it.<br></div><span><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 3, 2014 at 4:43 PM, Tammie Lister <span dir="ltr"><<a href="mailto:karmatosed@gmail.com" target="_blank">karmatosed@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Maybe an opportunity to include this in the guideline. We need to stop the use of imports :)</div></blockquote></div></div></span></div>
<br></div></div><span class="">_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></span></blockquote></div><br></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>