[theme-reviewers] Questions on my first review

[Peter Kakoma]

Hi Otto,
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.
That line, "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;

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.
I've re-uploaded the theme: http://themes.trac.wordpress.org/ticket/12964.
Springer, if you can, please take a look at it again. (Seeing as you've
already downloaded it and started the review :-) )

The credit link leads to my page: http:kakoma.ug, which I'm upgrading right
now. I can put something else if the current 'under construction' page is
an issue

Oh, and the 'read me', as Springer can attest, contains proper credits


On Tue, Jun 11, 2013 at 5:27 AM, Otto <otto at ottodestruct.com> wrote:

> On Mon, Jun 10, 2013 at 8:54 PM, J.J. Springer <jjgspringer at gmail.com>
> wrote:
> > 2. If I get any error of any sort, I assume that means that the theme is
> > automatically rejected. As soon as I run into an error, do I stop and
> mark
> > it rejected or keep going and try to find as many errors as possible? Is
> > there a point at which I should just stop and send it back?
>
> This is up to you. Additionally, I apologize for usurping your review
> and failing the theme, but that kind of ticked me off a bit.
>
> If you wish to do a full review and give the theme author more tips
> and things that he needs to check on, that's perfectly acceptable.
>
> > 3. The theme that I'm reviewing is a child theme of twentytwelve and
> > therefore doesn't have the "required" theme template files (missing
> > comments.php). Are child themes allowed? Is there some special way that
> I'm
> > supposed to be reviewing them?
>
> Child themes won't pass theme check, obviously, because of missing
> files and such. The review guidelines still stand though, and the
> child+parent must pass, as a whole, sort of thing. The parent is
> presumed to already have passed, since it's in the directory anyway.
>
>
> > 4. The person that I'm reviewing used a lot framework stuff that's MIT or
> > WTFPL licensed. Is that ok?
>
> Both of those are GPL-Compat, so they're fine.
>
>
> -Otto
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>



-- 
www.urbanlegendkampala.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20130611/932f7dcb/attachment.html>