[theme-reviewers] Questions on my first review

Chip Bennett chip at chipbennett.net
Tue Jun 11 22:42:32 UTC 2013


Google Analytics and sharing buttons are Plugin territory.


On Tue, Jun 11, 2013 at 5:00 PM, Peter Kakoma <kakomap at gmail.com> wrote:

> Hi Abhik,
> Thanks for the feedback; point 1 has been duly resolved...another
> implementation I picked from the framework author. We do agree on point 1.
> 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.
>
>
>
> On Tue, Jun 11, 2013 at 7:31 PM, Abhik Biswas <abhik at itsabhik.com> wrote:
>
>> Pete,
>> If I were reviewing your theme, I'd have rejected it straight away due to
>> two major issues.
>>
>> 1. Images should be resized using 'add_image_size', no external resizer
>> is allowed.
>> 2. Those social sharing buttons and Google Analytics integration. In my
>> opinion, they fall under plugin territory.
>>
>> But, it all depends on the reviewer.
>>
>>
>>
>> On Tue, Jun 11, 2013 at 9:55 AM, Peter Kakoma <kakomap at gmail.com> wrote:
>>
>>> Thanks Springer. I've added to the discussion
>>>
>>>
>>> On Tue, Jun 11, 2013 at 7:11 AM, J.J. Springer <jjgspringer at gmail.com>wrote:
>>>
>>>> Peter,
>>>> 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.
>>>>
>>>>
>>>> Otto,
>>>> Thanks for the speedy response! No worries about closing the ticket.
>>>> I'm new to the process so any help is appreciated.
>>>>
>>>>
>>>> Amy,
>>>> Thanks for the detailed answers. I just want to make sure that I do
>>>> everything right!
>>>>
>>>>
>>>> Thanks again everyone for the help! Can't wait for the next one!
>>>>
>>>> J.J.
>>>>
>>>>
>>>> On Mon, Jun 10, 2013 at 8:42 PM, Peter Kakoma <kakomap at gmail.com>wrote:
>>>>
>>>>> 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
>>>>>
>>>>> _______________________________________________
>>>>> theme-reviewers mailing list
>>>>> theme-reviewers at lists.wordpress.org
>>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>>
>>>>>
>>>>
>>>> _______________________________________________
>>>> theme-reviewers mailing list
>>>> theme-reviewers at lists.wordpress.org
>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>
>>>>
>>>
>>>
>>> --
>>> www.urbanlegendkampala.com
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>
>
> --
> www.urbanlegendkampala.com
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20130611/5c25b32c/attachment.html>


More information about the theme-reviewers mailing list