[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