[theme-reviewers] Theme review

Tue Apr 19 20:44:56 UTC 2011

Sure Tammie and it's all set http://themes.trac.wordpress.org/ticket/3487

<http://themes.trac.wordpress.org/ticket/3487>Thank you.
*----*
*Emil Uzelac* | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E:
emil at themeid.com | http://themeid.com
Make everything as simple as possible, but not simpler. - Albert Einstein

On Tue, Apr 19, 2011 at 3:40 PM, Tammie Lister <tlister at logicalbinary.com>wrote:

>  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
> ***********************************************
>
>
>
> _______________________________________________
> 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/4b47920c/attachment-0001.htm>