[theme-reviewers] Theme review

Tammie Lister tlister at logicalbinary.com
Tue Apr 19 20:40:42 UTC 2011 

I've done my second review here: http://themes.trac.wordpress.org/ticket/3478. Love to do another if one is available either WordPress or BuddyPress - I don't mind.

Just as a query is this the best method to ask for the next theme or is there another preferred method? My requests each time keep getting lost in noise which is fine if asking again is the way to do it. Just don't want to be the kid at back of class jumping up and down waving hand in air :)

Regards,
Tammie Lister

On Tuesday, 19 April 2011 at 17:53, theme-reviewers-request at lists.wordpress.org wrote: 
> 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. Re: Theme Request (rex posadas)
>  2. Re: Theme Request (Edward Caissie)
> 
> 
> ----------------------------------------------------------------------
> 
> Message: 1
> Date: Tue, 19 Apr 2011 09:50:43 -0700 (PDT)
> From: rex posadas <rexposadas at yahoo.com>
> Subject: Re: [theme-reviewers] Theme Request
> To: theme-reviewers at lists.wordpress.org
> Message-ID: <819188.53810.qm at web36507.mail.mud.yahoo.com>
> Content-Type: text/plain; charset="us-ascii"
> 
> @Cais
> 
> Thanks. I noted the items in my review comments.
> 
> How does the theme get officially approved?
> 
> Rex
> 
> 
> 
> ________________________________
> From: Edward Caissie <edward.caissie at gmail.com>
> To: theme-reviewers at lists.wordpress.org
> Sent: Tue, April 19, 2011 5:42:31 AM
> Subject: Re: [theme-reviewers] Theme Request
> 
> @rex - Please note these items on the ticket for the Theme author to be aware of 
> ...
> 
> ... every Theme has room to improvement, even the ones we see as "approved". 
> These would be good recommendations to add to your review comments.
> 
> 
> Cais.
> 
> 
> On Tue, Apr 19, 2011 at 12:30 AM, rex posadas <rexposadas at yahoo.com> wrote:
> 
> Thank you. 
> > 
> > 
> > I am done with my review and set the recommendation to "approve".
> > 
> > 
> > The two items listed below were the only ones that were questionable. Please let 
> > me know if I missed something. 
> > 
> > 
> > Thanks again, 
> > 
> > 
> > Rex
> ________________________________
> From: Chip Bennett <chip at chipbennett.net>
> > 
> > To: theme-reviewers at lists.wordpress.org
> > Sent: Mon, April 18, 2011 8:07:42 PM
> > 
> > Subject: Re: [theme-reviewers] Theme Request
> > 
> > 
> > I would note #1 as something to be fixed in the next revision. Use of 
> > "license.txt" explicitly is for (hopefully) some future integration/automation 
> > with Extend, and to try to create a de facto naming convention - but the primary 
> > intent is that the full-text license is declared. So, I wouldn't fail a review 
> > due to the naming inconsistency.
> > 
> > 
> > For #2: note it in the review, but this also isn't (alone) cause to fail a 
> > review.
> > 
> > 
> > Chip
> > 
> > 
> > On Mon, Apr 18, 2011 at 10:01 PM, rex posadas <rexposadas at yahoo.com> wrote:
> > 
> > To All. 
> > > 
> > > 
> > > Questions about the theme I am reviewing:
> > > 
> > > 
> > > 1) Is this a strict rule? The theme I am reviewing has "GPL.txt" instead of 
> > > "license.txt". Is it a cause for disapproval?
> > > 
> > > 
> > > Themes are required to declare their license explicitly, using one of the 
> > > following methods:
> > >  * Adding License and License URI header slugs to style.css
> > >  * Including a license.txt file with the Theme 2) A css line does not fare well 
> > > with the validators, but does not harm the layout. is it a cause for 
> > > disapproval? The css line is:
> > > 
> > > 
> > > border-radius: none;
> > > 
> > > 
> > > validators note "none" as an invalid value for that property.
> > > 
> > > 
> > > Thanks.
> > > 
> > > 
> > > Rex
> ________________________________
> From: Edward Caissie <edward.caissie at gmail.com>
> > > To: theme-reviewers at lists.wordpress.org
> > > Sent: Sat, April 16, 2011 4:42:46 PM
> > > Subject: Re: [theme-reviewers] Theme Request
> > > 
> > > 
> > > @rxn: Give this one a try http://themes.trac.wordpress.org/ticket/3367
> > > 
> > > 
> > > Cais.
> > > 
> > > 
> > > On Fri, Apr 15, 2011 at 1:24 PM, rex posadas <rexposadas at yahoo.com> wrote:
> > > 
> > > I just completed the steps to join the WP theme reviewers team, and request to
> > > > be assigned a theme for the review. My wordpress.org username is: rxn
> > > > 
> > > > Thanks and have a good day.
> > > > _______________________________________________
> > > > 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/20110419/d4cf502f/attachment-0001.htm>
> 
> ------------------------------
> 
> Message: 2
> Date: Tue, 19 Apr 2011 12:52:54 -0400
> From: Edward Caissie <edward.caissie at gmail.com>
> Subject: Re: [theme-reviewers] Theme Request
> To: theme-reviewers at lists.wordpress.org
> Message-ID: <BANLkTinhm2CBqhr06_5kRAizD1QTt-7FOQ at mail.gmail.com>
> Content-Type: text/plain; charset="iso-8859-1"
> 
> A WPTRT Admin (Chip, Simon, or myself) will generally go into Extend and
> mark it "Approved" there ... then it is live on the repository.
> 
> 
> Cais.
> 
> On Tue, Apr 19, 2011 at 12:50 PM, rex posadas <rexposadas at yahoo.com> wrote:
> 
> > @Cais
> > 
> > Thanks. I noted the items in my review comments.
> > 
> > How does the theme get officially approved?
> > 
> > Rex
> > 
> > 
> > ------------------------------
> > *From:* Edward Caissie <edward.caissie at gmail.com>
> > *To:* theme-reviewers at lists.wordpress.org
> > *Sent:* Tue, April 19, 2011 5:42:31 AM
> > 
> > *Subject:* Re: [theme-reviewers] Theme Request
> > 
> > @rex - Please note these items on the ticket for the Theme author to be
> > aware of ...
> > 
> > ... every Theme has room to improvement, even the ones we see as
> > "approved". These would be good recommendations to add to your review
> > comments.
> > 
> > 
> > Cais.
> > 
> > On Tue, Apr 19, 2011 at 12:30 AM, rex posadas <rexposadas at yahoo.com>wrote:
> > 
> > > Thank you.
> > > 
> > > I am done with my review and set the recommendation to "approve".
> > > 
> > > The two items listed below were the only ones that were questionable.
> > > Please let me know if I missed something.
> > > 
> > > Thanks again,
> > > 
> > > Rex
> > > 
> > > ------------------------------
> > > *From:* Chip Bennett <chip at chipbennett.net>
> > > 
> > > *To:* theme-reviewers at lists.wordpress.org
> > > *Sent:* Mon, April 18, 2011 8:07:42 PM
> > > 
> > > *Subject:* Re: [theme-reviewers] Theme Request
> > > 
> > > I would note #1 as something to be fixed in the next revision. Use of
> > > "license.txt" explicitly is for (hopefully) some future
> > > integration/automation with Extend, and to try to create a de facto naming
> > > convention - but the primary intent is that the full-text license is
> > > declared. So, I wouldn't fail a review due to the naming inconsistency.
> > > 
> > > For #2: note it in the review, but this also isn't (alone) cause to fail a
> > > review.
> > > 
> > > Chip
> > > 
> > > On Mon, Apr 18, 2011 at 10:01 PM, rex posadas <rexposadas at yahoo.com>wrote:
> > > 
> > > > To All.
> > > > 
> > > > Questions about the theme I am reviewing:
> > > > 
> > > > 1) Is this a strict rule? The theme I am reviewing has "GPL.txt" instead
> > > > of "license.txt". Is it a cause for disapproval?
> > > > 
> > > > Themes are *required* to declare their license explicitly, using one of
> > > > the following methods:
> > > > 
> > > >  - Adding *License* and *License URI* header slugs to style.css
> > > >  - Including a *license.txt* file with the Theme
> > > > 
> > > >  2) A css line does not fare well with the validators, but does not harm
> > > > the layout. is it a cause for disapproval? The css line is:
> > > > 
> > > > border-radius: none;
> > > > 
> > > > validators note "none" as an invalid value for that property.
> > > > 
> > > > Thanks.
> > > > 
> > > > Rex
> > > > *
> > > > *
> > > > *
> > > > *
> > > > 
> > > > ------------------------------
> > > > *From:* Edward Caissie <edward.caissie at gmail.com>
> > > > *To:* theme-reviewers at lists.wordpress.org
> > > > *Sent:* Sat, April 16, 2011 4:42:46 PM
> > > > *Subject:* Re: [theme-reviewers] Theme Request
> > > > 
> > > > @rxn: Give this one a try http://themes.trac.wordpress.org/ticket/3367
> > > > 
> > > > 
> > > > Cais.
> > > > 
> > > > On Fri, Apr 15, 2011 at 1:24 PM, rex posadas <rexposadas at yahoo.com>wrote:
> > > > 
> > > > > I just completed the steps to join the WP theme reviewers team, and
> > > > > request to
> > > > > be assigned a theme for the review. My wordpress.org username is:
> > > > > rxn
> > > > > 
> > > > > Thanks and have a good day.
> > > > > _______________________________________________
> > > > > 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
> > 
> > _______________________________________________
> > 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/20110419/025f4424/attachment.htm>
> 
> ------------------------------
> 
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
> 
> 
> End of theme-reviewers Digest, Vol 11, Issue 86
> ***********************************************
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/0059aa41/attachment-0001.htm>

More information about the theme-reviewers mailing list