<div dir="ltr"><div><div><div>Okay you're right, there are use cases where shortcodes should be allowed and are allowed as far as I remember. But most likeley they are used by theme developers to create user content. <br>
<br>I can see your point that a phrase which forbid them totally would be too hard, but then maybe just a phrase that Shortcodes and CPTs which generate user content are not allowed, because they stuck users to a certain theme. <br>
<br></div>Perhaps there is really no need for another rule, but a better explanation of the existing one would be nice. Just a small hint would be enough, which explains that theme authors have to pay attention by implementing shortcodes and CPTs, because they often slid quickly into plugin territority with their theme features. <br>
<br></div>Best Regards,<br></div>Thomas<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/4/30 Philip M. Hofer (Frumph) <span dir="ltr"><<a href="mailto:philip@frumph.net" target="_blank">philip@frumph.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">
<div dir="ltr">
<div style="font-size:12pt;font-family:'Calibri'">
<div>
<div style="font-size:small;font-style:normal;text-decoration:none;font-family:'Calibri';display:inline;font-weight:normal"><font face="Calibri" size="3">They stumble over it because it’s the reviewers
determination to whether or not they warrant being in a plugin and not in the
theme.</font></div></div>
<div>
<div style="font-size:small;font-style:normal;text-decoration:none;font-family:'Calibri';display:inline;font-weight:normal"> </div></div>
<div>
<div style="font-size:small;font-style:normal;text-decoration:none;font-family:'Calibri';display:inline;font-weight:normal">It’s
not a black and white.</div></div>
<div>
<div style="font-size:small;font-style:normal;text-decoration:none;font-family:'Calibri';display:inline;font-weight:normal"> </div></div>
<div>
<div style="font-size:small;font-style:normal;text-decoration:none;font-family:'Calibri';display:inline;font-weight:normal">Jeeze,
imagine a single plugin for a theme that does an output of theme content in a
shortcode that’s specific for that theme, .. uh what use is that ?</div></div>
<div>
<div style="font-size:small;font-style:normal;text-decoration:none;font-family:'Calibri';display:inline;font-weight:normal"> </div></div>
<div>
<div style="font-size:small;font-style:normal;text-decoration:none;font-family:'Calibri';display:inline;font-weight:normal">seriously.</div></div>
<div>
<div style="font-size:small;font-style:normal;text-decoration:none;font-family:'Calibri';display:inline;font-weight:normal"> </div></div>
<div>
<div style="font-size:small;font-style:normal;text-decoration:none;font-family:'Calibri';display:inline;font-weight:normal"></div>
<div style="FONT:10pt tahoma">
<div><font face="Calibri" size="3"></font> </div>
<div><font face="Calibri" size="3"></font> </div>
<div style="BACKGROUND:#f5f5f5">
<div><b>From:</b> <a title="contact@themezee.com" href="mailto:contact@themezee.com" target="_blank">Thomas from ThemeZee</a> </div>
<div><b>Sent:</b> Tuesday, April 30, 2013 2:07 AM</div>
<div><b>To:</b> <a title="theme-reviewers@lists.wordpress.org" href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>
</div>
<div><b>Subject:</b> Re: [theme-reviewers] Coding Standard</div></div></div>
<div> </div></div>
<div style="font-size:small;font-style:normal;text-decoration:none;font-family:'Calibri';display:inline;font-weight:normal"><div><div class="h5">
<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 style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div dir="ltr">This.
<div> </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> </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> </div>
<div>Thanks,</div>
<div> </div>
<div>Chip</div></div>
<div>
<div>
<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 style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">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>
<div> </div></div></div></div><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></blockquote></div>
<div> </div></div>
</div></div><p>
</p><hr><div class="im">
_______________________________________________<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><p></p></div></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>