<div dir="ltr">I see where Chip is coming from and I do agree. Ultimately, that option shouldn't be there at all; themes idealistically should bring offer nothing but a UI with options for styling that UI, and if there is any input left open to the user it should be nothing more than rich text content displayed in some section of the theme.<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 6:58 PM, <span dir="ltr"><<a href="mailto:theme-reviewers-request@lists.wordpress.org" target="_blank">theme-reviewers-request@lists.wordpress.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Send theme-reviewers mailing list submissions to<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<br>
To subscribe or unsubscribe via the World Wide Web, visit<br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
or, via email, send a message with subject or body 'help' to<br>
<a href="mailto:theme-reviewers-request@lists.wordpress.org">theme-reviewers-request@lists.wordpress.org</a><br>
<br>
You can reach the person managing the list at<br>
<a href="mailto:theme-reviewers-owner@lists.wordpress.org">theme-reviewers-owner@lists.wordpress.org</a><br>
<br>
When replying, please edit your Subject line so it is more specific<br>
than "Re: Contents of theme-reviewers digest..."<br>
<br>Today's Topics:<br>
<br>
1. Re: Fwd: Plugin territory (Josh Pollock)<br>
2. Re: Fwd: Plugin territory (Josh Pollock)<br>
<br><br>---------- Forwarded message ----------<br>From: Josh Pollock <<a href="mailto:jpollock412@gmail.com">jpollock412@gmail.com</a>><br>To: theme-reviewers <<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a>><br>
Cc: <br>Date: Mon, 17 Jun 2013 18:50:40 -0400<br>Subject: Re: [theme-reviewers] Fwd: Plugin territory<br><div dir="ltr">Its not instructing the user to add analytics code, its making it obvious to them that they can use this option to insert analytics code. By allowing theme options for header and footer scripts that make it clear they can be used for Google Analytics code or other uses, we're giving the end user an option. When they switch themes they can make sure that their new theme has these options, or turn to a plugin. I don't think you can go out on a limb and say that plugin is going to be better coded. Most likely it is if it came from the WordPress.org plugin repo, but it might have come from anywhere. You can't guarantee the quality of every random plugin on the internet.<div>
<br></div><div>I agree a plugin is the better way to do it. But then again, I wouldn't use a plugin, I'd just modify the theme, which I'd obviously version controlled so I wouldn't loose that modification when I updated the theme, but weren't not talking about users like me here. We're making a valid option, less obvious to the end user, and I don't really see how we're protecting them by doing so.</div>
<div><div><div><br></div></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 6:27 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</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">No, I'm saying that a Theme option to allow the user to insert arbitrary scripts or custom CSS is perfectly fine, but that a Theme *instructing* the user to use that option to insert analytics code is not, because it is functionally equivalent to the Theme providing an option intentionally for inserting analytics code.<div>
<br></div><div>We're not making their lives *more* complicated; we've making their lives *less* complicated - because analytics and SEO become a matter of "set and forget", regardless of what Theme is being used. (For the typical user, the question isn't *if* that user will switch Themes in the future, but rather *when* that user will switch Themes.) Further, I'll go out on a limb and say that a Plugin intended specifically for analytics code and/or SEO is going to be *better* coded, and *more* likely to be kept up-to-date, than analogous options in a given Theme.</div>
</div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 6:11 PM, Josh Pollock <span dir="ltr"><<a href="mailto:jpollock412@gmail.com" target="_blank">jpollock412@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">Chip-<div><br></div><div>Am I understanding you correctly that if a theme has an option called "add header script", that is OK, but if it has an option called "add analytics code to header" that is not OK, even though they do the exact same thing? That doesn't make sense.</div>
<div><br></div><div>This requirement is making users add one more plugin to worry about updating, checking for incompatibilities with, etc. We're talking about a class of users that doesn't know to/ isin't comfortable doing something as simple as putting the tracking code in a function hooked to wp_header/footer or just cut and paste it into header.php. These aren't people who can evaluate a plugin to see if it is adding the analytics the right way or not... Why are we making their lives more complicated instead of judging the "add analytics" option the same way as the forbidden "add header script" option, which we allow as long as it is implemented properly?</div>
<span><font color="#888888">
<div><br></div><div>-Josh</div></font></span></div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 2:09 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</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">I would require the text to be changed, if it says "add your analytics code", since adding analytics code is Plugin territory.</div>
<div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 1:53 PM, Srikanth Koneru <span dir="ltr"><<a href="mailto:tskk79@gmail.com" target="_blank">tskk79@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"><div>some are labelled "add your analytics code" and some just "add your header/footer scripts".<br>
</div><div>I approved them without pointing it out as i wasn't sure of the rule.<br>
</div></div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 11:18 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</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">How do you know that the end user will use them for that?</div><div><div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 1:36 PM, Srikanth Koneru <span dir="ltr"><<a href="mailto:tskk79@gmail.com" target="_blank">tskk79@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">because they are used to add analytics scripts or similar to header/footer?<br><br></div><div>
<div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 11:04 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</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">Why would they?<div><br></div><div>(Not being sarcastic; I'm honestly asking.)</div></div><div>
<div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 1:31 PM, Srikanth Koneru <span dir="ltr"><<a href="mailto:tskk79@gmail.com" target="_blank">tskk79@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">do textarea field's meant for header/footer scripts fall under plugin territory?<br></div><div>
<div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 10:57 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</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">Themes are always required to conform to the guidelines as current when the Theme is submitted. Guidelines can and do change, and previous reviews can and do miss required criteria. I would note all of the required issues, and hold the ticket open to allow for developer response.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote"><div><div>On Mon, Jun 17, 2013 at 1:03 PM, devcorn <span dir="ltr"><<a href="mailto:wp@devcorn.com" target="_blank">wp@devcorn.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>
<div dir="ltr">hi,<div><br><div class="gmail_quote"><div dir="ltr"><div>I was reviewing one theme and it has so many.. many things which comes under plugin territory shortcodes, SEO, download/upload... etc</div><div><br>
</div>
<div>But the theme was previously approved, actually always approved... so I don't know what should I do with it. I don't want to disappoint the author , I can approve it with note to fix all these stuff in next version, but to my surprise. I downloaded and test old version and they had all of it.. so I don't know if I missed something as I joined WTRT last week only.</div>
<div><br></div><div><br></div><div>Thanks</div><div>Ash</div></div>
<br></div><br></div></div>
<br></div></div>_______________________________________________<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><br></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><br></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><br></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><br></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><br></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><br></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><br></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><br></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><br></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><br></div>
<br><br>---------- Forwarded message ----------<br>From: Josh Pollock <<a href="mailto:jpollock412@gmail.com">jpollock412@gmail.com</a>><br>To: theme-reviewers <<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a>><br>
Cc: <br>Date: Mon, 17 Jun 2013 18:57:46 -0400<br>Subject: Re: [theme-reviewers] Fwd: Plugin territory<br><div dir="ltr">@Philip- I sent my last message for reading yours. You've given several examples of when the no plugin option might be better than the plugin option, which is in general better. Your last example especially illustrates how this can hurt developers and users. If a user is looking specifically for themes that can add the tracking code, and list it as a feature--because they need that, but need to limit their plugin count--they are not going to choose a theme that doesn't specifically say it has an option to add header/ footer scripts such as tracking codes, even though it does. Now the user missed out on a good theme, and the developer's theme got used one less time. Many of us are giving away themes in hopes of the end user paying us for an upsell version, theme customization, or a fancy child theme. Every time we loose a user of our free theme, we loose a potential customer for our services that we charge for.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 17, 2013 at 6:45 PM, Philip M. Hofer (Frumph) <span dir="ltr"><<a href="mailto:philip@frumph.net" target="_blank">philip@frumph.net</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">
<div dir="ltr">
<div style="font-size:12pt;font-family:'Calibri'">
<div>the Google Analyticator plugin is so awesome; there’s no comparison.</div>
<div> </div>
<div>.. on the fence with SEO; I haven’t seen one that is done right.. yoast is
close; but the biggest problem is ones like all-in-one generate the data for
each and every thing on every page load; which is bad; that’s a lot of
processing going on. I think to be done right it needs to
incorporate saving SEO data within meta fields and other things and not always
regenerated on each page load.. </div>
<div> </div>
<div>Also, there’s the ‘load’ of the plugin which hasn’t been discussed; each
plugin contains a higher memory footprint for each and every plugin that is
loaded. While little things like the google analytics code
being so simple and all; having it as an increase in the memory footprint
(execution time on server; mem used etc) might actually be a benefit to
the user.</div>
<div> </div>
<div>Plugin (saves in data, generates extra resource for each plugin used) .. or
one single line storing the Google Code which executes with the theme.. uh...
yeah.</div>
<div> </div>
<div>There are a couple services that some clients of mine are on that those
users cannot have more then 4-5 plugins active because the footprint of
wordpress is higher then the amount of memory those services give in available
processing/memory.</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 style="FONT:10pt tahoma">
<div><font face="Calibri" size="3"></font> </div>
<div style="BACKGROUND:#f5f5f5">
<div><b>From:</b> <a title="chip@chipbennett.net" href="mailto:chip@chipbennett.net" target="_blank">Chip Bennett</a> </div>
<div><b>Sent:</b> Monday, June 17, 2013 3:27 PM</div>
<div><b>To:</b> <a title="theme-reviewers@lists.wordpress.org" href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">[theme-reviewers]</a> </div>
<div><b>Subject:</b> Re: [theme-reviewers] Fwd: Plugin
territory</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>
<div dir="ltr">No, I'm saying that a Theme option to allow the user to insert
arbitrary scripts or custom CSS is perfectly fine, but that a Theme
*instructing* the user to use that option to insert analytics code is not,
because it is functionally equivalent to the Theme providing an option
intentionally for inserting analytics code.
<div> </div>
<div>We're not making their lives *more* complicated; we've making their lives
*less* complicated - because analytics and SEO become a matter of "set and
forget", regardless of what Theme is being used. (For the typical user, the
question isn't *if* that user will switch Themes in the future, but rather
*when* that user will switch Themes.) Further, I'll go out on a limb and say
that a Plugin intended specifically for analytics code and/or SEO is going to be
*better* coded, and *more* likely to be kept up-to-date, than analogous options
in a given Theme.</div></div>
<div class="gmail_extra"><br><br>
<div class="gmail_quote">On Mon, Jun 17, 2013 at 6:11 PM, Josh Pollock <span dir="ltr"><<a href="mailto:jpollock412@gmail.com" target="_blank">jpollock412@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">
<div dir="ltr">Chip-
<div> </div>
<div>Am I understanding you correctly that if a theme has an option called
"add header script", that is OK, but if it has an option called "add analytics
code to header" that is not OK, even though they do the exact same thing? That
doesn't make sense.</div>
<div> </div>
<div>This requirement is making users add one more plugin to worry about
updating, checking for incompatibilities with, etc. We're talking about a
class of users that doesn't know to/ isin't comfortable doing something as
simple as putting the tracking code in a function hooked to wp_header/footer
or just cut and paste it into header.php. These aren't people who can evaluate
a plugin to see if it is adding the analytics the right way or not... Why are
we making their lives more complicated instead of judging the "add analytics"
option the same way as the forbidden "add header script" option, which we
allow as long as it is implemented properly?</div><span><font color="#888888">
<div> </div>
<div>-Josh</div></font></span></div>
<div>
<div>
<div class="gmail_extra"><br><br>
<div class="gmail_quote">On Mon, Jun 17, 2013 at 2:09 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div dir="ltr">I would require the text to be changed, if it says "add your
analytics code", since adding analytics code is Plugin territory.</div>
<div>
<div>
<div class="gmail_extra"><br><br>
<div class="gmail_quote">On Mon, Jun 17, 2013 at 1:53 PM, Srikanth Koneru
<span dir="ltr"><<a href="mailto:tskk79@gmail.com" target="_blank">tskk79@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">
<div dir="ltr">
<div>some are labelled "add your analytics code" and some just "add your
header/footer scripts".<br></div>
<div>I approved them without pointing it out as i wasn't sure of the
rule.<br></div></div>
<div>
<div>
<div class="gmail_extra"><br><br>
<div class="gmail_quote">On Mon, Jun 17, 2013 at 11:18 PM, Chip Bennett
<span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div dir="ltr">How do you know that the end user will use them for
that?</div>
<div>
<div>
<div class="gmail_extra"><br><br>
<div class="gmail_quote">On Mon, Jun 17, 2013 at 1:36 PM, Srikanth Koneru
<span dir="ltr"><<a href="mailto:tskk79@gmail.com" target="_blank">tskk79@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">
<div dir="ltr">because they are used to add analytics scripts or similar
to header/footer?<br><br></div>
<div>
<div>
<div class="gmail_extra"><br><br>
<div class="gmail_quote">On Mon, Jun 17, 2013 at 11:04 PM, Chip Bennett
<span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div dir="ltr">Why would they?
<div> </div>
<div>(Not being sarcastic; I'm honestly asking.)</div></div>
<div>
<div>
<div class="gmail_extra"><br><br>
<div class="gmail_quote">On Mon, Jun 17, 2013 at 1:31 PM, Srikanth
Koneru <span dir="ltr"><<a href="mailto:tskk79@gmail.com" target="_blank">tskk79@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">
<div dir="ltr">do textarea field's meant for header/footer scripts
fall under plugin territory?<br></div>
<div>
<div>
<div class="gmail_extra"><br><br>
<div class="gmail_quote">On Mon, Jun 17, 2013 at 10:57 PM, Chip
Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div dir="ltr">Themes are always required to conform to the
guidelines as current when the Theme is submitted. Guidelines
can and do change, and previous reviews can and do miss required
criteria. I would note all of the required issues, and hold the
ticket open to allow for developer response.</div>
<div class="gmail_extra"><br><br>
<div class="gmail_quote">
<div>
<div>On Mon, Jun 17, 2013 at 1:03 PM, devcorn <span dir="ltr"><<a href="mailto:wp@devcorn.com" target="_blank">wp@devcorn.com</a>></span>
wrote:<br></div></div>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div>
<div>
<div dir="ltr">hi,
<div>
<div> </div>
<div class="gmail_quote">
<div dir="ltr">
<div>I was reviewing one theme and it has so many.. many
things which comes under plugin territory shortcodes,
SEO, download/upload... etc</div>
<div> </div>
<div>But the theme was previously approved, actually always
approved... so I don't know what should I do with it. I don't
want to disappoint the author , I can approve it with note to
fix all these stuff in next version, but to my surprise. I
downloaded and test old version and they had all of it.. so I
don't know if I missed something as I joined WTRT last week
only.</div>
<div> </div>
<div> </div>
<div>Thanks</div>
<div>Ash</div></div>
<div> </div></div>
<div> </div></div></div>
<div> </div></div></div>_______________________________________________<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><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><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><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><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><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><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><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><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>
_______________________________________________<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" 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><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><br clear="all"><br>-- <br><div dir="ltr">Jonathan Weatherhead <br><br>Software Consultant<br><div><blog> <a href="http://planetjon.ca/" target="_blank">http://planetjon.ca</a><br><portfolio> <a href="http://jonathanweatherhead.com/" target="_blank">http://jonathanweatherhead.com</a></div>
</div>
</div>