<div dir="ltr">Plugin territory is a not-approve condition.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 11, 2013 at 6:45 PM, Harish <span dir="ltr"><<a href="mailto:me@harishchouhan.com" target="_blank">me@harishchouhan.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-IN" link="blue" vlink="purple"><div><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Hello Chip,<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">There are many themes in the repo that have Google Analytics integration that could be activated from Theme Options. <u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Should a theme be rejected because it offers options for Google Analytics and Sharing or social buttons? <u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#404040"><u></u> <u></u></span></p>
<p class="MsoNormal" style="line-height:150%"><span style="font-size:11.0pt;line-height:150%;font-family:"Calibri","sans-serif";color:#404040">Regards,<u></u><u></u></span></p><p class="MsoNormal" style="line-height:150%">
<span style="font-size:11.0pt;line-height:150%;font-family:"Calibri","sans-serif";color:#404040">Harish</span><span style="font-size:10.0pt;line-height:150%;font-family:"Calibri","sans-serif";color:#0d0d0d"> <b><u></u><u></u></b></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri","sans-serif"">From:</span></b><span lang="EN-US" style="font-size:11.0pt;font-family:"Calibri","sans-serif""> theme-reviewers [mailto:<a href="mailto:theme-reviewers-bounces@lists.wordpress.org" target="_blank">theme-reviewers-bounces@lists.wordpress.org</a>] <b>On Behalf Of </b>Chip Bennett<br>
<b>Sent:</b> Wed 12 June 13 04:13 AM<br><b>To:</b> [theme-reviewers]<br><b>Subject:</b> Re: [theme-reviewers] Questions on my first review<u></u><u></u></span></p><div><div class="h5"><p class="MsoNormal"><u></u> <u></u></p>
<div><p class="MsoNormal">Google Analytics and sharing buttons are Plugin territory.<u></u><u></u></p></div><div><p class="MsoNormal" style="margin-bottom:12.0pt"><u></u> <u></u></p><div><p class="MsoNormal">On Tue, Jun 11, 2013 at 5:00 PM, Peter Kakoma <<a href="mailto:kakomap@gmail.com" target="_blank">kakomap@gmail.com</a>> wrote:<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm"><div><p class="MsoNormal">Hi Abhik,<u></u><u></u></p><div><p class="MsoNormal">Thanks for the feedback; point 1 has been duly resolved...another implementation I picked from the framework author. We do agree on point 1. <u></u><u></u></p>
</div><div><p class="MsoNormal" style="margin-bottom:12.0pt">Point 2 though, as far as I know, isn't grounds to reject a theme. Plugin/non-plugin territory, to the best of my knowledge, has been debated ad nauseam and the jury is still out. <u></u><u></u></p>
</div></div><div><div><div><p class="MsoNormal" style="margin-bottom:12.0pt"><u></u> <u></u></p><div><p class="MsoNormal">On Tue, Jun 11, 2013 at 7:31 PM, Abhik Biswas <<a href="mailto:abhik@itsabhik.com" target="_blank">abhik@itsabhik.com</a>> wrote:<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm"><div><div><p class="MsoNormal"><span style="font-family:"Verdana","sans-serif"">Pete,<u></u><u></u></span></p>
</div><div><p class="MsoNormal" style="margin-bottom:12.0pt"><span style="font-family:"Verdana","sans-serif"">If I were reviewing your theme, I'd have rejected it straight away due to two major issues.<u></u><u></u></span></p>
</div><div><p class="MsoNormal"><span style="font-family:"Verdana","sans-serif"">1. Images should be resized using 'add_image_size', no external resizer is allowed.<u></u><u></u></span></p></div>
<div><p class="MsoNormal" style="margin-bottom:12.0pt"><span style="font-family:"Verdana","sans-serif"">2. Those social sharing buttons and Google Analytics integration. In my opinion, they fall under plugin territory.<u></u><u></u></span></p>
</div><div><p class="MsoNormal"><span style="font-family:"Verdana","sans-serif"">But, it all depends on the reviewer.<u></u><u></u></span></p></div><div><p class="MsoNormal"><span style="font-family:"Verdana","sans-serif""><u></u> <u></u></span></p>
</div></div><div><div><div><p class="MsoNormal" style="margin-bottom:12.0pt"><u></u> <u></u></p><div><p class="MsoNormal">On Tue, Jun 11, 2013 at 9:55 AM, Peter Kakoma <<a href="mailto:kakomap@gmail.com" target="_blank">kakomap@gmail.com</a>> wrote:<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm"><div><p class="MsoNormal">Thanks Springer. I've added to the discussion<u></u><u></u></p></div>
<div><div><div><p class="MsoNormal" style="margin-bottom:12.0pt"><u></u> <u></u></p><div><p class="MsoNormal">On Tue, Jun 11, 2013 at 7:11 AM, J.J. Springer <<a href="mailto:jjgspringer@gmail.com" target="_blank">jjgspringer@gmail.com</a>> wrote:<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm"><div><div><p class="MsoNormal" style="margin-bottom:12.0pt">Peter, <br>I have updated your ticket with some other things that I found while I was reviewing the theme. I hope that they are helpful. Your ReadMe does have the proper credits, so thank you for that. :)  I just wasn't sure what licenses were allowed.<br>
<br><u></u><u></u></p></div><div><p class="MsoNormal" style="margin-bottom:12.0pt">Otto, <br>Thanks for the speedy response! No worries about closing the ticket. I'm new to the process so any help is appreciated. <br>
<br><u></u><u></u></p></div><div><p class="MsoNormal" style="margin-bottom:12.0pt">Amy,<br>Thanks for the detailed answers. I just want to make sure that I do everything right!<br><br><u></u><u></u></p></div><div><p class="MsoNormal" style="margin-bottom:12.0pt">
Thanks again everyone for the help! Can't wait for the next one!<u></u><u></u></p></div><div><p class="MsoNormal"><span style="color:#888888">J.J.<u></u><u></u></span></p></div></div><div><div><div><p class="MsoNormal" style="margin-bottom:12.0pt">
<u></u> <u></u></p><div><p class="MsoNormal">On Mon, Jun 10, 2013 at 8:42 PM, Peter Kakoma <<a href="mailto:kakomap@gmail.com" target="_blank">kakomap@gmail.com</a>> wrote:<u></u><u></u></p><blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm">
<div><p class="MsoNormal">Hi Otto,<u></u><u></u></p><div><p class="MsoNormal">The theme in question is mine; only issue it turns out after re-reading all the tips here is the base_64 encoding. I customized a  very popular and very user-friendly framework -the Slightly Modified Options Framework. Very big themes use it.<br>
That l<span style="font-size:10.5pt;font-family:"Arial","sans-serif";color:#444444">ine, "100% safe - ignore theme check nag", which I know got you :-) was added by the Framework author. I took his word for it...seeing as the framework is used everywhere. I figured several themes in the repository already use it. I'm very aware of the base64 encoding rule;  <br>
<br>I've removed the option altogether; the encoding was being used to allow users import/export/transfer their settings. Will find another way of doing it...and will make changes to the framework repository so hopefully, the author can clean that up.</span><u></u><u></u></p>
</div><div><p class="MsoNormal"><span style="font-size:10.5pt;font-family:"Arial","sans-serif";color:#444444">I've re-uploaded the theme: </span><a href="http://themes.trac.wordpress.org/ticket/12964" target="_blank">http://themes.trac.wordpress.org/ticket/12964</a>. <u></u><u></u></p>
</div><div><p class="MsoNormal">Springer, if you can, please take a look at it again. (Seeing as you've already downloaded it and started the review :-) )<u></u><u></u></p></div><div><p class="MsoNormal"><u></u> <u></u></p>
</div><div><p class="MsoNormal">The credit link leads to my page: http:<a href="http://kakoma.ug" target="_blank">kakoma.ug</a>, which I'm upgrading right now. I can put something else if the current 'under construction' page is an issue<u></u><u></u></p>
</div><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal">Oh, and the 'read me', as Springer can attest, contains proper credits<u></u><u></u></p></div></div><div><div><div><p class="MsoNormal" style="margin-bottom:12.0pt">
<u></u> <u></u></p><div><p class="MsoNormal">On Tue, Jun 11, 2013 at 5:27 AM, Otto <<a href="mailto:otto@ottodestruct.com" target="_blank">otto@ottodestruct.com</a>> wrote:<u></u><u></u></p><blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-right:0cm">
<div><p class="MsoNormal">On Mon, Jun 10, 2013 at 8:54 PM, J.J. Springer <<a href="mailto:jjgspringer@gmail.com" target="_blank">jjgspringer@gmail.com</a>> wrote:<u></u><u></u></p></div><div><p class="MsoNormal" style="margin-bottom:12.0pt">
> 2. If I get any error of any sort, I assume that means that the theme is<br>> automatically rejected. As soon as I run into an error, do I stop and mark<br>> it rejected or keep going and try to find as many errors as possible? Is<br>
> there a point at which I should just stop and send it back?<u></u><u></u></p></div><p class="MsoNormal">This is up to you. Additionally, I apologize for usurping your review<br>and failing the theme, but that kind of ticked me off a bit.<br>
<br>If you wish to do a full review and give the theme author more tips<br>and things that he needs to check on, that's perfectly acceptable.<u></u><u></u></p><div><p class="MsoNormal" style="margin-bottom:12.0pt"><br>
> 3. The theme that I'm reviewing is a child theme of twentytwelve and<br>> therefore doesn't have the "required" theme template files (missing<br>> comments.php). Are child themes allowed? Is there some special way that I'm<br>
> supposed to be reviewing them?<u></u><u></u></p></div><p class="MsoNormal">Child themes won't pass theme check, obviously, because of missing<br>files and such. The review guidelines still stand though, and the<br>
child+parent must pass, as a whole, sort of thing. The parent is<br>presumed to already have passed, since it's in the directory anyway.<u></u><u></u></p><div><p class="MsoNormal" style="margin-bottom:12.0pt"><br><br>
> 4. The person that I'm reviewing used a lot framework stuff that's MIT or<br>> WTFPL licensed. Is that ok?<u></u><u></u></p></div><p class="MsoNormal">Both of those are GPL-Compat, so they're fine.<br>
<span style="color:#888888"><br><br>-Otto</span><u></u><u></u></p><div><div><p class="MsoNormal">_______________________________________________<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><u></u><u></u></p></div></div></blockquote></div><p class="MsoNormal"><br>
<br clear="all"><u></u><u></u></p><div><p class="MsoNormal"><u></u> <u></u></p></div></div></div><p class="MsoNormal"><span style="color:#888888">-- <br><a href="http://www.urbanlegendkampala.com" target="_blank">www.urbanlegendkampala.com</a></span><u></u><u></u></p>
</div><p class="MsoNormal" style="margin-bottom:12.0pt"><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><u></u><u></u></p></blockquote></div><p class="MsoNormal"><u></u> <u></u></p>
</div></div></div><p class="MsoNormal" style="margin-bottom:12.0pt"><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><u></u><u></u></p></blockquote></div><p class="MsoNormal"><br><br clear="all">
<u></u><u></u></p><div><p class="MsoNormal"><u></u> <u></u></p></div><p class="MsoNormal">-- <br><a href="http://www.urbanlegendkampala.com" target="_blank">www.urbanlegendkampala.com</a><u></u><u></u></p></div></div></div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><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><u></u><u></u></p></blockquote></div><p class="MsoNormal"><u></u> <u></u></p>
</div></div></div><p class="MsoNormal" style="margin-bottom:12.0pt"><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><u></u><u></u></p></blockquote></div><p class="MsoNormal"><br><br clear="all">
<u></u><u></u></p><div><p class="MsoNormal"><u></u> <u></u></p></div><p class="MsoNormal">-- <br><a href="http://www.urbanlegendkampala.com" target="_blank">www.urbanlegendkampala.com</a><u></u><u></u></p></div></div></div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><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><u></u><u></u></p></blockquote></div><p class="MsoNormal"><u></u> <u></u></p>
</div></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>