[theme-reviewers] General Question of theme review

Tue May 21 00:29:47 UTC 2013

Validation isn't required, and honestly isn't all that important -- it
can be a useful diagnostic tool when a lot of things seem to be
displaying wrong, but aside from the weird edge case of XHTML-strict
it doesn't actually make a difference.

On Mon, May 20, 2013 at 8:26 PM, Harish <me at harishchouhan.com> wrote:
> - When reviewing a theme, how much importance should be placed to HTML & CSS
> Validation? Should the theme have zero errors before its approved?
>
> - I have reviewed my first theme ticket, would appreciate if anyone could
> assign me another one.
> http://make.wordpress.org/themes/about/trac-ticket-request-queue/trac-ticket
> -request-queue-mar-2013/
>
>
>
> Regards,
> Harish
>
>
> -----Original Message-----
> From: theme-reviewers [mailto:theme-reviewers-bounces at lists.wordpress.org]
> On Behalf Of theme-reviewers-request at lists.wordpress.org
> Sent: Tue 21 May 13 05:41 AM
> To: theme-reviewers at lists.wordpress.org
> Subject: theme-reviewers Digest, Vol 36, Issue 79
>
> Send theme-reviewers mailing list submissions to
>         theme-reviewers at lists.wordpress.org
>
> To subscribe or unsubscribe via the World Wide Web, visit
>         http://lists.wordpress.org/mailman/listinfo/theme-reviewers
> or, via email, send a message with subject or body 'help' to
>         theme-reviewers-request at lists.wordpress.org
>
> You can reach the person managing the list at
>         theme-reviewers-owner at lists.wordpress.org
>
> When replying, please edit your Subject line so it is more specific than
> "Re: Contents of theme-reviewers digest..."
>
>
> Today's Topics:
>
>    1. the_excerpt() in the loop (Stephen Cui)
>    2. Re: the_excerpt() in the loop (Chip Bennett)
>    3. Re: Push Live Request (Chip Bennett)
>    4. Re: Push Live Request (Daniel)
>
>
> ----------------------------------------------------------------------
>
> Message: 1
> Date: Mon, 20 May 2013 18:43:09 -0500
> From: Stephen Cui <scui2005 at gmail.com>
> To: theme-reviewers at lists.wordpress.org
> Subject: [theme-reviewers] the_excerpt() in the loop
> Message-ID:
>         <CANfF7QXSBT6_4cyc8WKmZ=EGKtHxGMWSnjiGarO-er-5B6-aiQ at mail.gmail.com>
> Content-Type: text/plain; charset="iso-8859-1"
>
> Hi,
>
> Just a observation: I have reviewed several themes for past weeks. It seems
> that the_excerpt() become the only choice in the main lopp (e.g index.php)
> for may theme authors.. I know it is not required to use the_content() in
> the the guideline. I just want to know what is your thoughts on this trend?
>
> Stephen
> -------------- next part --------------
> An HTML attachment was scrubbed...
> URL:
> <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20130520/8
> cb5e603/attachment-0001.html>
>
> ------------------------------
>
> Message: 2
> Date: Mon, 20 May 2013 19:51:39 -0400
> From: Chip Bennett <chip at chipbennett.net>
> To: "[theme-reviewers]" <theme-reviewers at lists.wordpress.org>
> Subject: Re: [theme-reviewers] the_excerpt() in the loop
> Message-ID:
>         <CAPdLKqc=wfm=jLKcVR31ANJzwTU=_52MXurTA2q85yRe8G8rrA at mail.gmail.com>
> Content-Type: text/plain; charset="iso-8859-1"
>
> Developer prerogative. As long as the single-post view displays
> the_content(), then archive index views are free to use either
> the_content() or the_excerpt().
>
>
> On Mon, May 20, 2013 at 7:43 PM, Stephen Cui <scui2005 at gmail.com> wrote:
>
>> Hi,
>>
>> Just a observation: I have reviewed several themes for past weeks. It
>> seems that the_excerpt() become the only choice in the main lopp (e.g
>> index.php) for may theme authors.. I know it is not required to use
>> the_content() in the the guideline. I just want to know what is your
>> thoughts on this trend?
>>
>> Stephen
>>
>>
>> _______________________________________________
>> 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/20130520/3
> 2c21496/attachment-0001.html>
>
> ------------------------------
>
> Message: 3
> Date: Mon, 20 May 2013 20:07:40 -0400
> From: Chip Bennett <chip at chipbennett.net>
> To: "[theme-reviewers]" <theme-reviewers at lists.wordpress.org>
> Subject: Re: [theme-reviewers] Push Live Request
> Message-ID:
>         <CAPdLKqehYbnH45VB5KOONzJmocOPX738UiKjVuc6GquHq2tsvg at mail.gmail.com>
> Content-Type: text/plain; charset="iso-8859-1"
>
> Post (with poll) to discuss the idea of having Priority #1 Queue tickets
> reserved for Admin review:
> http://make.wordpress.org/themes/2013/05/21/should-priority-1-queue-be-reser
> ved-for-admin-review/
>
>
> On Mon, May 20, 2013 at 4:01 PM, Chip Bennett <chip at chipbennett.net> wrote:
>
>> I'll write up something this evening on Make/Themes, and solicit
>> feedback on the idea.
>>
>>
>> On Mon, May 20, 2013 at 3:45 PM, Josh Pollock
> <jpollock412 at gmail.com>wrote:
>>
>>> @chip I honk that's a good idea, to make priority #1 queue reviewed
>>> by admins only. It should speed up the queue for new themes, as it
>>> would let the rest of us concentrate on that backlog. What if we try
>>> it, and if it results in a backlog of previously approved themes, or
>>> takes up too much of the admins' time, we assign 1 or 2 experienced
>>> reviewers to help them out with those reviews?
>>>
>>> On Monday, May 20, 2013, Zulfikar Nore wrote:
>>>
>>>> @Justin - that's another idea worth keeping in mind, thanks for
>>>> sharing
>>>> - I'm learning new tricks as I go along :)
>>>>
>>>> @Chip that's a great idea on "Only Admins reviewing priority 1 que"
>>>> - the only concern would Admins being over used allover the place
>>>> i.e. review priority 1 & push live, check other reviewers reviews &
>>>> push those live, allocate tickets and maybe training new reviewers?
>>>> But if it can be worked then I'm all for it.
>>>>
>>>> ------------------------------
>>>> Date: Mon, 20 May 2013 12:34:38 -0500
>>>> From: justin at justintadlock.com
>>>> To: theme-reviewers at lists.wordpress.org
>>>> Subject: Re: [theme-reviewers] Push Live Request
>>>>
>>>> Also, you can download the newer version directly from .ORG.  If I
>>>> have a theme update, I change the download links on my site to the
>>>> newer version, even if it hasn't been through the review process yet.
>>>>
>>>> On 5/20/2013 9:35 AM, Paul Appleyard wrote:
>>>>
>>>> Zulf,
>>>>
>>>> Perhaps you can email those desperate users the update directly?
>>>> Most themes I develop are also generally available from a git hub or
>>>> bitbucket repository for direct download.
>>>>
>>>> Paul Appleyard
>>>>
>>>> On 21/05/2013 12:32 AM, Zulfikar Nore wrote:
>>>>
>>>> Hello Admins,
>>>>
>>>> I know its not been 7 days since approval but can I be cheeky and
>>>> ask for these two to pushed live as the updates resolve an issue a
>>>> couple of the users are having..
>>>> http://themes.trac.wordpress.org/ticket/12469
>>>> http://themes.trac.wordpress.org/ticket/12479
>>>>
>>>> Thanks,
>>>> Zulf.
>>>>
>>>>
>>>> _______________________________________________
>>>> theme-reviewers mailing
>>>> listtheme-reviewers at lists.wordpress.orghttp://lists.wordpress.org/ma
>>>> ilman/listinfo/theme-reviewers
>>>>
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> theme-reviewers mailing
>>>> listtheme-reviewers at lists.wordpress.orghttp://lists.wordpress.org/ma
>>>> ilman/listinfo/theme-reviewers
>>>>
>>>>
>>>>
>>>> _______________________________________________ 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/20130520/5
> f91249c/attachment-0001.html>
>
> ------------------------------
>
> Message: 4
> Date: Tue, 21 May 2013 10:10:34 +1000
> From: Daniel <danielx386 at gmail.com>
> To: theme-reviewers at lists.wordpress.org
> Subject: Re: [theme-reviewers] Push Live Request
> Message-ID:
>         <CAJ5Hi2i-ssUmY5saAv0+bbwBD8hzrs2Waw=3MbyfATU9b-_R1g at mail.gmail.com>
> Content-Type: text/plain; charset="utf-8"
>
> Love the idea and already voted :)
>
> On Tue, May 21, 2013 at 10:07 AM, Chip Bennett <chip at chipbennett.net> wrote:
>
>> Post (with poll) to discuss the idea of having Priority #1 Queue
>> tickets reserved for Admin review:
>>
>> http://make.wordpress.org/themes/2013/05/21/should-priority-1-queue-be
>> -reserved-for-admin-review/
>>
>>
>> On Mon, May 20, 2013 at 4:01 PM, Chip Bennett <chip at chipbennett.net>wrote:
>>
>>> I'll write up something this evening on Make/Themes, and solicit
>>> feedback on the idea.
>>>
>>>
>>> On Mon, May 20, 2013 at 3:45 PM, Josh Pollock
> <jpollock412 at gmail.com>wrote:
>>>
>>>> @chip I honk that's a good idea, to make priority #1 queue reviewed
>>>> by admins only. It should speed up the queue for new themes, as it
>>>> would let the rest of us concentrate on that backlog. What if we try
>>>> it, and if it results in a backlog of previously approved themes, or
>>>> takes up too much of the admins' time, we assign 1 or 2 experienced
>>>> reviewers to help them out with those reviews?
>>>>
>>>> On Monday, May 20, 2013, Zulfikar Nore wrote:
>>>>
>>>>>  @Justin - that's another idea worth keeping in mind, thanks for
>>>>> sharing - I'm learning new tricks as I go along :)
>>>>>
>>>>> @Chip that's a great idea on "Only Admins reviewing priority 1 que"
>>>>> - the only concern would Admins being over used allover the place
>>>>> i.e. review priority 1 & push live, check other reviewers reviews &
>>>>> push those live, allocate tickets and maybe training new reviewers?
>>>>> But if it can be worked then I'm all for it.
>>>>>
>>>>>  ------------------------------
>>>>> Date: Mon, 20 May 2013 12:34:38 -0500
>>>>> From: justin at justintadlock.com
>>>>> To: theme-reviewers at lists.wordpress.org
>>>>> Subject: Re: [theme-reviewers] Push Live Request
>>>>>
>>>>> Also, you can download the newer version directly from .ORG.  If I
>>>>> have a theme update, I change the download links on my site to the
>>>>> newer version, even if it hasn't been through the review process yet.
>>>>>
>>>>> On 5/20/2013 9:35 AM, Paul Appleyard wrote:
>>>>>
>>>>> Zulf,
>>>>>
>>>>> Perhaps you can email those desperate users the update directly?
>>>>> Most themes I develop are also generally available from a git hub
>>>>> or bitbucket repository for direct download.
>>>>>
>>>>> Paul Appleyard
>>>>>
>>>>> On 21/05/2013 12:32 AM, Zulfikar Nore wrote:
>>>>>
>>>>> Hello Admins,
>>>>>
>>>>> I know its not been 7 days since approval but can I be cheeky and
>>>>> ask for these two to pushed live as the updates resolve an issue a
>>>>> couple of the users are having..
>>>>> http://themes.trac.wordpress.org/ticket/12469
>>>>> http://themes.trac.wordpress.org/ticket/12479
>>>>>
>>>>> Thanks,
>>>>> Zulf.
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> theme-reviewers mailing
>>>>> listtheme-reviewers at lists.wordpress.orghttp://lists.wordpress.org/m
>>>>> ailman/listinfo/theme-reviewers
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> theme-reviewers mailing
>>>>> listtheme-reviewers at lists.wordpress.orghttp://lists.wordpress.org/m
>>>>> ailman/listinfo/theme-reviewers
>>>>>
>>>>>
>>>>>
>>>>> _______________________________________________ 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
>>>>
>>>>
>>>
>>
>> _______________________________________________
>> 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/20130521/1
> 08ff582/attachment.html>
>
> ------------------------------
>
> Subject: Digest Footer
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
> ------------------------------
>
> End of theme-reviewers Digest, Vol 36, Issue 79
> ***********************************************
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers