[theme-reviewers] theme-reviewers Digest, Vol 11, Issue 92

Shawn Grundy smgrundy at live.com
Wed Apr 20 05:09:22 UTC 2011


done with both...
Shawn

From: emil at themeid.com
Date: Tue, 19 Apr 2011 22:02:54 -0500
To: theme-reviewers at lists.wordpress.org
Subject: Re: [theme-reviewers] theme-reviewers Digest, Vol 11, Issue 92

Two tickets assigned to: smgrundy




http://themes.trac.wordpress.org/ticket/3499http://themes.trac.wordpress.org/ticket/3500


Thanks,

Emil----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 9:56 PM, Shawn Grundy <smgrundy at live.com> wrote:







I will take a theme or two Emil if you have them!
username: smgrundy

From: emil at themeid.com
Date: Tue, 19 Apr 2011 18:27:59 -0500


To: theme-reviewers at lists.wordpress.org
Subject: Re: [theme-reviewers] theme-reviewers Digest, Vol 11, Issue 92



No worries and 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 6:27 PM, Tammie Lister <karmatosed at gmail.com> wrote:




Thanks will review that one tomorrow.



Regards

Tammie Lister



On 20 Apr 2011, at 00:13, 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 review completed (Emil Uzelac)

>

>

> ----------------------------------------------------------------------

>

> Message: 1

> Date: Tue, 19 Apr 2011 18:13:12 -0500

> From: Emil Uzelac <emil at themeid.com>

> Subject: Re: [theme-reviewers] Theme review completed

> To: theme-reviewers at lists.wordpress.org

> Message-ID: <BANLkTimk8CPe8txXC5OG-Fm=3y7jwfPR5A at mail.gmail.com>

> Content-Type: text/plain; charset="iso-8859-1"

>

> Sure http://themes.trac.wordpress.org/ticket/3498

> *----*

> *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 6:11 PM, Tammie Lister <tlister at logicalbinary.com>wrote:

>

>>  I've completed another theme review tonight and would like to review

>> another please tomorrow taking the feedback on board of extra things I

>> should be looking for.  Again happy either BuddyPress or WordPress.

>>

>> http://themes.trac.wordpress.org/ticket/3487 was one did tonight.

>>

>> Regards,

>> Tammie Lister

>>

>> On Tuesday, 19 April 2011 at 23:58,

>> 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: @smgrundy [WordPress Themes] #3563: THEME: For

>> Women-Female - 1.1 (Shawn Grundy)

>> 2. Re: @smgrundy [WordPress Themes] #3563: THEME: For

>> Women-Female - 1.1 (Emil Uzelac)

>> 3. After 3367 - request (rex posadas)

>> 4. Re: Another ticket (Curtis McHale)

>> 5. Re: Another ticket (Chip Bennett)

>>

>>

>> ----------------------------------------------------------------------

>>

>> Message: 1

>> Date: Tue, 19 Apr 2011 14:21:38 -0700

>> From: Shawn Grundy <smgrundy at live.com>

>> Subject: Re: [theme-reviewers] @smgrundy [WordPress Themes] #3563:

>> THEME: For Women-Female - 1.1

>> To: <theme-reviewers at lists.wordpress.org>

>> Message-ID: <COL102-W23982B423D7C0CA1596795AE900 at phx.gbl>

>> Content-Type: text/plain; charset="iso-8859-1"

>>

>>

>> done

>>

>> From: emil at themeid.com

>> Date: Tue, 19 Apr 2011 15:50:22 -0500

>> To: theme-reviewers at lists.wordpress.org

>> Subject: Re: [theme-reviewers] @smgrundy [WordPress Themes] #3563: THEME:

>> For Women-Female - 1.1

>>

>> ----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:49 PM, WordPress Themes <

>> theme-reviewers at lists.wordpress.org> wrote:

>>

>>

>> #3563: THEME: For Women-Female - 1.1

>>

>> ------------------------+-----------------------------------

>>

>> Reporter: Stilisten | Owner: smgrundy

>>

>> Type: theme | Status: assigned

>>

>> Resolution: | Keywords: theme-for-women-female

>>

>> ------------------------+-----------------------------------

>>

>> Changes (by emiluzelac):

>>

>>

>>

>> * owner: => smgrundy

>>

>> * status: new => assigned

>>

>>

>>

>>

>>

>> Comment:

>>

>>

>>

>> @smgrundy since you reviewed first time here it is once again. This should

>>

>> be good to go as the author fixed all of the items.

>>

>>

>>

>> --

>>

>> Ticket URL: <http://themes.trac.wordpress.org/ticket/3563#comment:1>

>>

>> WordPress Themes <http://themes.trac.wordpress.org/>

>>

>> WordPress.org Theme Directory Reviews

>>

>>

>>

>>

>> _______________________________________________

>> 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/e2e4a631/attachment-0001.htm





>>>

>>

>> ------------------------------

>>

>> Message: 2

>> Date: Tue, 19 Apr 2011 16:29:11 -0500

>> From: Emil Uzelac <emil at themeid.com>

>> Subject: Re: [theme-reviewers] @smgrundy [WordPress Themes] #3563:

>> THEME: For Women-Female - 1.1

>> To: theme-reviewers at lists.wordpress.org

>> Message-ID: <BANLkTikKrPxMtJrueT+CK-pt4BfWN1nJpA at mail.gmail.com>

>> Content-Type: text/plain; charset="iso-8859-1"

>>

>> 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 4:21 PM, Shawn Grundy <smgrundy at live.com> wrote:

>>

>> done

>>

>> ------------------------------

>> From: emil at themeid.com

>> Date: Tue, 19 Apr 2011 15:50:22 -0500

>> To: theme-reviewers at lists.wordpress.org

>> Subject: Re: [theme-reviewers] @smgrundy [WordPress Themes] #3563: THEME:

>> For Women-Female - 1.1

>>

>>

>>

>> *----*

>> *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:49 PM, WordPress Themes <

>> theme-reviewers at lists.wordpress.org> wrote:

>>

>> #3563: THEME: For Women-Female - 1.1

>> ------------------------+-----------------------------------

>> Reporter: Stilisten | Owner: smgrundy

>> Type: theme | Status: assigned

>> Resolution: | Keywords: theme-for-women-female

>> ------------------------+-----------------------------------

>> Changes (by emiluzelac):

>>

>> * owner: => smgrundy

>> * status: new => assigned

>>

>>

>> Comment:

>>

>> @smgrundy since you reviewed first time here it is once again. This should

>> be good to go as the author fixed all of the items.

>>

>> --

>> Ticket URL: <http://themes.trac.wordpress.org/ticket/3563#comment:1>

>> WordPress Themes <http://themes.trac.wordpress.org/>

>> WordPress.org Theme Directory Reviews

>>

>>

>>

>> _______________________________________________ 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/5999840f/attachment-0001.htm





>>>

>>

>> ------------------------------

>>

>> Message: 3

>> Date: Tue, 19 Apr 2011 15:33:07 -0700 (PDT)

>> From: rex posadas <rexposadas at yahoo.com>

>> Subject: [theme-reviewers] After 3367 - request

>> To: theme-reviewers at lists.wordpress.org

>> Message-ID: <969894.75830.qm at web36504.mail.mud.yahoo.com>

>> Content-Type: text/plain; charset="us-ascii"

>>

>> Hello,

>>

>> I just reviewed 3367, I am ready for another theme.

>>

>> username: rxn

>>

>> Thank You.

>>

>> Rex

>> -------------- next part --------------

>> An HTML attachment was scrubbed...

>> URL: <

>> http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/96396658/attachment-0001.htm





>>>

>>

>> ------------------------------

>>

>> Message: 4

>> Date: Tue, 19 Apr 2011 15:44:52 -0700

>> From: Curtis McHale <curtis at curtismchale.ca>

>> Subject: Re: [theme-reviewers] Another ticket

>> To: theme-reviewers at lists.wordpress.org

>> Message-ID: <BANLkTi==o+EbjW9CVjQ3BOcsmrZ4MAVXaQ at mail.gmail.com>

>> Content-Type: text/plain; charset="iso-8859-1"

>>

>> The theme I'm reviewing has JS that detects the browser version and tells

>> them to update (http://browser-update.org/). There is not a theme option

>> to

>> remove this function or information in the description.

>>

>> What is the theme review stance on this feature?

>>

>> There are also some list styles missing in comments. I don't think this is

>> a

>> reason for failure but should be noted and requested for update on the next

>> version. Is that correct?

>>

>> The theme options are strictly a page that suggests the user donate if they

>> like the theme. I know that credit links are okay but I'm not sure about

>> this instance.

>>

>> The theme calls loop-archive.php through get_template_part but the file

>> doesn't actually exist. Fortunately it defaults to loop.php but I'm

>> wondering how this would be addressed. The theme works but I would think

>> this should be corrected in the next version either by just using loop.php

>> or by adding the file that is being called. Is that correct?

>>

>> The theme doesn't support post_formats. I thought that they were supposed

>> to

>> at least declare it and support whichever they picked. I don't see this

>> information on the Theme Review page so is this assumption incorrect?

>>

>> Curtis McHale

>> PH: 604.751.3482

>> http://www.curtismchale.ca

>> Linkedin: http://www.linkedin.com/in/curtismchale

>> Twitter: twitter.com/curtismchale

>> -------------- next part --------------

>> An HTML attachment was scrubbed...

>> URL: <

>> http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/9b218d3d/attachment-0001.htm





>>>

>>

>> ------------------------------

>>

>> Message: 5

>> Date: Tue, 19 Apr 2011 17:58:21 -0500

>> From: Chip Bennett <chip at chipbennett.net>

>> Subject: Re: [theme-reviewers] Another ticket

>> To: theme-reviewers at lists.wordpress.org

>> Message-ID: <BANLkTi=Z_Qb3E9msQQNLhMAr70_Hrk5CmA at mail.gmail.com>

>> Content-Type: text/plain; charset="iso-8859-1"

>>

>> Comments inline...

>>

>> On Tue, Apr 19, 2011 at 5:44 PM, Curtis McHale <curtis at curtismchale.ca

>>> wrote:

>>

>> The theme I'm reviewing has JS that detects the browser version and tells

>> them to update (http://browser-update.org/). There is not a theme option

>> to remove this function or information in the description.

>>

>> What is the theme review stance on this feature?

>>

>>

>> Incredibly annoying, but nothing prohibits it. I would *require* a mention

>> of the "feature" in Theme documentation (readme.txt).

>>

>>

>> There are also some list styles missing in comments. I don't think this is

>> a reason for failure but should be noted and requested for update on the

>> next version. Is that correct?

>>

>>

>> It depends on how egregious the style omission is. If lists within comments

>> are inheriting the comment-list styling, I would fail for that. If they

>> merely don't have proper indenting/bullets, I would perhaps say *required*

>> in next revision.

>>

>>

>> The theme options are strictly a page that suggests the user donate if they

>> like the theme. I know that credit links are okay but I'm not sure about

>> this instance.

>>

>>

>> Also nothing prohibiting this. But I would balk at calling such a page an

>> "Options" page. You could perhaps require the page to be titled "Theme

>> Documentation" (etc.) rather than "Theme Options".

>>

>>

>> The theme calls loop-archive.php through get_template_part but the file

>> doesn't actually exist. Fortunately it defaults to loop.php but I'm

>> wondering how this would be addressed. The theme works but I would think

>> this should be corrected in the next version either by just using loop.php

>> or by adding the file that is being called. Is that correct?

>>

>>

>> That could be for Child-Theme support (allowing e.g. a Child Theme to

>> include "loop-archive.php", and have it included). Due to the inherent

>> callback nature of the function, this is a correct implementation.

>>

>>

>> The theme doesn't support post_formats. I thought that they were supposed

>> to at least declare it and support whichever they picked. I don't see this

>> information on the Theme Review page so is this assumption incorrect?

>>

>>

>> Post Formats are not required to be implemented. However, if the Theme has

>> custom loops for any of the Post Format types, then it is *required* to

>> support the core feature.

>>

>> Chip

>> -------------- next part --------------

>> An HTML attachment was scrubbed...

>> URL: <

>> http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/01dbdb66/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 90

>> ***********************************************

>>

>>

>>

>> _______________________________________________

>> 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/e7f725e6/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 92

> ***********************************************

_______________________________________________

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/a8d7dce6/attachment-0001.htm>


More information about the theme-reviewers mailing list