<div dir="ltr"><div><div><div><div>Hi everybody,<br><br></div>in my opinion the Guidelines should clearly point out that Shortcodes and Custom Post Types are not allowed to use in themes. This is a point where both reviewers and theme authors often stumble over.<br>
<br></div>I know we have the passage: "Since the purpose of Themes is to define the presentation of user
content, Themes must not be used to define the generation of user
content, or to define Theme-independent site options or functionality." <br><br>But most people misinterpret this passage or overread it or maybe don't realize what it means - I don't know - but we had the discussion about Shortcodes and CPTs a lot lateley. I had to ask about it a few weeks ago and I have contributed some themes to the directory a long time before. <br>
<br>The Guidelines already contain a clear phrase that TimThumb is not allowed, I would suggest that there is a similiar phrase in the Guidelines which forbid shortcodes and CPTs.<br><br></div>Best Regards,<br></div>Thomas<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/4/29 Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">This.<div><br></div><div>The Guidelines - i.e. the things that are *required* - are in one place only: the Theme Review Codex page:</div><div><a href="http://codex.wordpress.org/Theme_Review" target="_blank">http://codex.wordpress.org/Theme_Review</a><br>
</div><div><br></div><div>The Make/Themes site is used to discuss the guidelines; if anything is formalized, it will be added to the Codex page. There are things that we occasionally miss, so if you have any specific Guidelines that were decided on the Make/Themes site, but not added to the Codex, please bring them up, and we'll make sure the Codex is accurate.</div>
<div><br></div><div>Thanks,</div><div><br></div><div>Chip</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 29, 2013 at 5:35 PM, Amy Hendrix <span dir="ltr"><<a href="mailto:sabreuse@gmail.com" target="_blank">sabreuse@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If that's a common occurrence (and I haven't been reviewing enough<br>
lately to know how common it really is), then that's a MAJOR problem.<br>
The Make site is where changes to the guidelines are *discussed*, but<br>
it's the nature of such discussions that not all points that are<br>
discussed are actually adopted as guidelines.<br>
<br>
To put it bluntly, reviewers, please DO NOT say things are required if<br>
they aren't actually required in the Review Guidelines. Passing the<br>
review process is difficult already; we absolutely need to have a<br>
clear and consistently enforced set of guidelines. There are enough<br>
complaints out there that the guidelines are arbitrary and constantly<br>
changing without people saying things are required when they're not.<br>
<div><div><br>
On Mon, Apr 29, 2013 at 5:29 PM, Srikanth Koneru <<a href="mailto:tskk79@gmail.com" target="_blank">tskk79@gmail.com</a>> wrote:<br>
> can't recall any instantly, but reviewers usually point to make site and<br>
> state that its a required issue even tough its not mentioned on org<br>
> guidelines.<br>
><br>
><br>
> On Tue, Apr 30, 2013 at 2:56 AM, Chip Bennett <<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>> wrote:<br>
>><br>
>> What policies, specifically?<br>
>><br>
>><br>
>> On Mon, Apr 29, 2013 at 5:25 PM, Srikanth Koneru <<a href="mailto:tskk79@gmail.com" target="_blank">tskk79@gmail.com</a>> wrote:<br>
>>><br>
>>> A few policies are formulated on make site but they don't make it to<br>
>>> guidelines on .org<br>
>>> so these policies are only recommended then?<br>
>>><br>
>>><br>
>>> On Tue, Apr 30, 2013 at 2:53 AM, Chip Bennett <<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>><br>
>>> wrote:<br>
>>>><br>
>>>> Yes. At the moment, the Coding and CSS standards are only *recommended*.<br>
>>>> Making them *required* may be a bridge we cross at some point, but we're not<br>
>>>> there right now.<br>
>>>><br>
>>>><br>
>>>> On Mon, Apr 29, 2013 at 5:07 PM, Emil Uzelac <<a href="mailto:emil@uzelac.me" target="_blank">emil@uzelac.me</a>> wrote:<br>
>>>>><br>
>>>>> If this does not cause any issues, I would say recommended.<br>
>>>>><br>
>>>>> Emil<br>
>>>>><br>
>>>>><br>
>>>>> On Mon, Apr 29, 2013 at 3:53 PM, Stephen Cui <<a href="mailto:scui2005@gmail.com" target="_blank">scui2005@gmail.com</a>><br>
>>>>> wrote:<br>
>>>>>><br>
>>>>>> Hi,<br>
>>>>>><br>
>>>>>> During the theme review, my theme has a "required issue" for next<br>
>>>>>> version:<br>
>>>>>><br>
>>>>>> There are inconsistency in spacing around parenthesis throughout the<br>
>>>>>> theme. Recommend to read the WordPress? PHP coding standards.<br>
>>>>>><br>
>>>>>><br>
>>>>>> <a href="http://make.wordpress.org/core/handbook/coding-standards/php/#space-usage" target="_blank">http://make.wordpress.org/core/handbook/coding-standards/php/#space-usage</a><br>
>>>>>><br>
>>>>>> I have no problem to make such changes as it is good practice. I also<br>
>>>>>> take the chance to revisit the code.<br>
>>>>>><br>
>>>>>> My question is " is it required?" or "just recommended".<br>
>>>>>><br>
>>>>>> Regards<br>
>>>>>><br>
>>>>>> Stephen<br>
>>>>>><br>
>>>>>> _______________________________________________<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>
>>>>><br>
>>>>><br>
>>>>> _______________________________________________<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>
>>>><br>
>>>><br>
>>>> _______________________________________________<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>
>>><br>
>>><br>
>>> _______________________________________________<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>
>><br>
>><br>
>> _______________________________________________<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>
><br>
><br>
> _______________________________________________<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>
_______________________________________________<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>
</div></div></blockquote></div><br></div>
</div></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>