[theme-reviewers] recomended vs required

[Srikanth Koneru]

Maybe we should not reopen tickets unless its license and/or security
issue, small issues even a few can easily fixed downstream. Reopening will
only add to reviewer workload, it will be kicked back to original reviewer
and if that reviewer is missing, it will only add to theme author's
frustration.

On Tue, Oct 14, 2014 at 2:29 AM, David Chandra <david at shellcreeper.com>
wrote:

> Thank you Tammie,
> I understand that the recommendation by the theme reviewer is for the best.
> I do understand that personal/professional knowledge vs theme guidelines
> can sometimes in a blurry line.
>
> once i got a reviewer said:
> It's required for all functions in functions.php to have function_exist
> check.
> (which is not even in guidelines)
>
> I wish to theme reviewer to better understand the guidelines.
> so they can better inform to theme author about their theme,
> which are "recommended" which are "required" and maybe their personal
> "opinion" and/or "suggestion" to make the theme better.
>
> Of course, the theme authors can receive their suggestion in better
> perspective.
>
> -- David.
>
>
>
>
> On Tue, Oct 14, 2014 at 3:32 AM, Tammie Lister <karmatosed at gmail.com>
> wrote:
>
>>  Nobody is trying to single you out, nobody is making an example of you,
>> nobody is witch hunting anyone. Everyone just wants to make your theme the
>> best it can be and work with you to get your theme live. Just consider this
>> when interacting and don't assume someone's actions. Asking and having an
>> open discussion with a person always works better.
>>
>> Thanks
>> Tammie
>>
>>
>> On 13/10/2014 21:09, David Chandra wrote:
>>
>>     Thank you Justin.
>>
>>  and I'm sorry to all if my comment about "witch hunt target" is a too
>> "strong".
>>  I didn't mean it that way. I'm truly understand what the reviewer
>> suggested is good. very good.
>>
>>  I responded that way because i feel, maybe as example:
>>
>> I want to create a passport, and I completed all the legal document,
>> and finally after hours and hours of waiting in line I take photo shoot,
>>  and all I need is to wait for the passport.
>>  But suddenly they said that
>>  "Sir, you need to cut your hair, i think your passport photo will look
>> better"
>> and i respond: "No thanks, I think i look fine."
>>  and they respond "Are you sure you want to look like this in your photo?
>> You need to cut your hair. Your photo is going to be there for 5 years."
>>
>>   of course my respond would be "what the..." i need to clarify about
>> the passport registration procedure.
>>
>>  -- David.
>>
>>
>>
>>
>>
>> On Tue, Oct 14, 2014 at 2:53 AM, Justin Tadlock <justin at justintadlock.com
>> > wrote:
>>
>>>   do theme reviewer now start to not approving themes because not doing
>>>> the recomended even already do all required in guidelines?
>>>>
>>>  Recommended guidelines are only recommended.  It's often a good idea
>>> to do them, but you are not required to do so.
>>>
>>>> if the theme already pass all the required in the guidelines do theme
>>>> reviewer need to make theme author to do extra recommended guide?
>>>>
>>>  It'd be great if reviewers would mention these things, but there's no
>>> reason a reviewer should hold a theme back from being approved for things
>>> marked recommended.
>>>
>>>  Also, not all guidelines that are recommended are good for every
>>> theme.  For example, the custom background feature is a recommended
>>> feature, but it doesn't make sense to use that feature in all themes.  In
>>> fact, it can be a downright horrible feature to use with some theme
>>> designs.  Basically, recommended items are things we want you to be aware
>>> of.  We want you to think about whether these additions would be a good
>>> thing for your theme.
>>>
>>>
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>
>>
>> _______________________________________________
>> theme-reviewers mailing listtheme-reviewers at lists.wordpress.orghttp://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>> --
>> Thanks
>> Tammie Lister
>>
>>
>> _______________________________________________
>> 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20141014/4458ed86/attachment.html>