[theme-reviewers] General Question of theme review

Chip Bennett chip at chipbennett.net
Tue May 21 01:00:04 UTC 2013


HTML/CSS validation is only a concern if something breaks visually. Per the
Guidelines, HTML/CSS validation is only *recommended*.


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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20130520/140ce9dd/attachment-0001.html>


More information about the theme-reviewers mailing list