<div dir="ltr">Hi Otto,<div style>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<font color="#444444" face="sans-serif"><span style="font-size:14px;line-height:19px">ine, </span></font><span style="color:rgb(68,68,68);font-family:sans-serif;font-size:14px;line-height:19px">"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></div>
<div style><span style="color:rgb(68,68,68);font-family:sans-serif;font-size:14px;line-height:19px">I've re-uploaded the theme: </span><a href="http://themes.trac.wordpress.org/ticket/12964">http://themes.trac.wordpress.org/ticket/12964</a>. </div>
<div style>Springer, if you can, please take a look at it again. (Seeing as you've already downloaded it and started the review :-) )</div><div style><br></div><div style>The credit link leads to my page: http:<a href="http://kakoma.ug">kakoma.ug</a>, which I'm upgrading right now. I can put something else if the current 'under construction' page is an issue</div>
<div style><br></div><div style>Oh, and the 'read me', as Springer can attest, contains proper credits</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 11, 2013 at 5:27 AM, Otto <span dir="ltr"><<a href="mailto:otto@ottodestruct.com" target="_blank">otto@ottodestruct.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Mon, Jun 10, 2013 at 8:54 PM, J.J. Springer <<a href="mailto:jjgspringer@gmail.com">jjgspringer@gmail.com</a>> wrote:<br>
</div><div class="im">> 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?<br>
<br>
</div>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.<br>
<div class="im"><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?<br>
<br>
</div>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.<br>
<div class="im"><br>
<br>
> 4. The person that I'm reviewing used a lot framework stuff that's MIT or<br>
> WTFPL licensed. Is that ok?<br>
<br>
</div>Both of those are GPL-Compat, so they're fine.<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
-Otto<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<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>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><a href="http://www.urbanlegendkampala.com" target="_blank">www.urbanlegendkampala.com</a><br>
</div>