[theme-reviewers] theme-reviewers Digest, Vol 11, Issue 56
Tammie Lister
tlister at logicalbinary.com
Mon Apr 18 10:11:09 UTC 2011
Hi, thought about time I poked my head up and got involved too. My username is karmatosed on WordPress.
>
> I'll put myself forward for any BuddyPress ones but can also do WordPress ones if required. Don't know but sounds like you've got a bit of a backlog on BuddyPress ones? I'm a theme designer who does both BuddyPress and pure WordPress themes.
Regards,
Tammie Lister
On Monday, 18 April 2011 at 04:35, 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: Join the team (Emil Uzelac)
> 2. BuddyPress themes? (Doug Stewart)
> 3. Re: BuddyPress themes? (Emil Uzelac)
> 4. Please assign a theme from the Theme Review Trac Queue for
> username smgrundy at live.com (Shawn Grundy)
> 5. Where can I learn more about theme security (myazalea at hotmail.com)
>
>
> ----------------------------------------------------------------------
>
> Message: 1
> Date: Sun, 17 Apr 2011 21:04:54 -0500
> From: Emil Uzelac <emil at themeid.com>
> Subject: Re: [theme-reviewers] Join the team
> To: theme-reviewers at lists.wordpress.org
> Message-ID: <BANLkTi=N5TzbQh2C81-2pBr=7jSoF4kQHg at mail.gmail.com>
> Content-Type: text/plain; charset="iso-8859-1"
>
> menu is OK, widgetized menu is OK too ;)
> *----*
> *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 Sun, Apr 17, 2011 at 7:54 PM, Chip Bennett <chip at chipbennett.net> wrote:
>
> > I actually disagree regarding the "Menu" Widget - maybe. :)
> >
> > If the Theme simply provides a Widgetized sidebar, and no "Navigation Menu"
> > markup, then this approach is fine. However, if the Theme has what would
> > reasonably considered to be the design intent of a Navigation Menu, then
> > that implementation must support the core Nav Menu feature.
> >
> > Basically: Themes are not *required* to provide Navigation Menus, but if
> > they *do* provide a Navigation Menu, then it must support the core
> > functionality.
> >
> > Chip
> >
> >
> > On Sun, Apr 17, 2011 at 7:13 PM, Emil Uzelac <emil at themeid.com> wrote:
> >
> > > 1. The theme uses the text domain of Twenty Ten in a few spots. If I
> > > remember correctly from lurking on the list this is a reason for failure of
> > > the theme.
> > > *Yes, Twenty Ten should be changed to i.e. silverville*
> > > **
> > > 2. They use a widget for the menu and instruct the user to add the Pages
> > > widget. It seems to me that it's a much better option to use a WordPress
> > > menu which defaults to list pages. I don't believe this is an issue for a
> > > fail but probably a good suggestion as an update to the user.
> > > *This part is just fine. As long as there are options for menu and being
> > > fully functional.*
> > > **
> > > 3. The CSS fails on the background-color since it's value is set to 'none'
> > > when it should be set to 'transparent'. Would this typically be a reason for
> > > failure if all other CSS is okay?
> > > *background-color: none; is invalid and it should be transparent.
> > > Recommendation would be background: none; or background-image: none; and not
> > > many people use the last one anymore. It's shortcoded via background.*
> > > **
> > > 4. There are a number of other 'recommended' items in Theme Check but
> > > these are not a reason to fail a theme correct?
> > > *Recommended isn't required, if recommendations don't cause problems, no
> > > need to fail.
> > > *
> > > 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 Sun, Apr 17, 2011 at 7:01 PM, Curtis McHale <curtis at curtismchale.ca>wrote:
> > >
> > > > I have a few questions about my first theme review.
> > > >
> > > > 1. The theme uses the text domain of Twenty Ten in a few spots. If I
> > > > remember correctly from lurking on the list this is a reason for failure of
> > > > the theme.
> > > > 2. They use a widget for the menu and instruct the user to add the Pages
> > > > widget. It seems to me that it's a much better option to use a WordPress
> > > > menu which defaults to list pages. I don't believe this is an issue for a
> > > > fail but probably a good suggestion as an update to the user.
> > > > 3. The CSS fails on the background-color since it's value is set to
> > > > 'none' when it should be set to 'transparent'. Would this typically be a
> > > > reason for failure if all other CSS is okay?
> > > > 4. There are a number of other 'recommended' items in Theme Check but
> > > > these are not a reason to fail a theme correct?
> > > >
> > > > I could probably go further in to the theme but I wanted to double check
> > > > on the text domain item before I did.
> > > >
> > > > Curtis McHale
> > > > PH: 604.751.3482
> > > > http://www.curtismchale.ca
> > > > Linkedin: http://www.linkedin.com/in/curtismchale
> > > > Twitter: twitter.com/curtismchale
> > > >
> > > >
> > > >
> > > > On Sat, Apr 16, 2011 at 4:39 PM, Edward Caissie <
> > > > edward.caissie at gmail.com> wrote:
> > > >
> > > > > Here ya go: http://themes.trac.wordpress.org/ticket/3320
> > > > > This will get you started ...
> > > > >
> > > > >
> > > > > Cais.
> > > > >
> > > > > On Sat, Apr 16, 2011 at 7:15 PM, Curtis McHale <curtis at curtismchale.ca>wrote:
> > > > >
> > > > > > Figured I'd finally get off my butt and join the team after looking at
> > > > > > the list for a few months. My trac name is curtismchale
> > > > > >
> > > > > > Curtis McHale
> > > > > > PH: 604.751.3482
> > > > > > http://www.curtismchale.ca
> > > > > > Linkedin: http://www.linkedin.com/in/curtismchale
> > > > > > Twitter: twitter.com/curtismchale
> > > > > >
> > > > > >
> > > > > > _______________________________________________
> > > > > > 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
> >
> > _______________________________________________
> > 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/20110417/2bfcde18/attachment-0001.htm>
>
> ------------------------------
>
> Message: 2
> Date: Sun, 17 Apr 2011 23:17:51 -0400
> From: Doug Stewart <zamoose at gmail.com>
> Subject: [theme-reviewers] BuddyPress themes?
> To: theme-reviewers at lists.wordpress.org
> Message-ID: <BANLkTinFHy-X_z-MT4PUnfDQQ8js95o9fQ at mail.gmail.com>
> Content-Type: text/plain; charset=ISO-8859-1
>
> Just how far behind is the BP review queue? I've been doing some BP
> work and might be able to take a look.
>
> Username is ZaMoose.
>
> --
> -Doug
>
>
> ------------------------------
>
> Message: 3
> Date: Sun, 17 Apr 2011 22:21:58 -0500
> From: Emil Uzelac <emil at themeid.com>
> Subject: Re: [theme-reviewers] BuddyPress themes?
> To: theme-reviewers at lists.wordpress.org
> Message-ID: <BANLkTik3+SrXTKZPqPw48A7iec6=g-_05w at mail.gmail.com>
> Content-Type: text/plain; charset="iso-8859-1"
>
> one in queue and thanks http://themes.trac.wordpress.org/ticket/3515
> *----*
> *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 Sun, Apr 17, 2011 at 10:17 PM, Doug Stewart <zamoose at gmail.com> wrote:
>
> > Just how far behind is the BP review queue? I've been doing some BP
> > work and might be able to take a look.
> >
> > Username is ZaMoose.
> >
> > --
> > -Doug
> > _______________________________________________
> > 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/20110417/ad26ab56/attachment-0001.htm>
>
> ------------------------------
>
> Message: 4
> Date: Sun, 17 Apr 2011 20:26:43 -0700
> From: Shawn Grundy <smgrundy at live.com>
> Subject: [theme-reviewers] Please assign a theme from the Theme Review
> Trac Queue for username smgrundy at live.com
> To: <theme-reviewers at lists.wordpress.org>
> Message-ID: <COL102-W52B1D6E3D4B364C1BE079AE910 at phx.gbl>
> Content-Type: text/plain; charset="iso-8859-1"
>
>
> Please assign a theme from the Theme Review Trac Queue for username smgrundy at live.com
> -------------- next part --------------
> An HTML attachment was scrubbed...
> URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110417/0a9a1abf/attachment-0001.htm>
>
> ------------------------------
>
> Message: 5
> Date: Mon, 18 Apr 2011 05:34:21 +0200
> From: myazalea at hotmail.com <myazalea at hotmail.com>
> Subject: [theme-reviewers] Where can I learn more about theme security
> To: theme-reviewers at lists.wordpress.org
> Message-ID: <BLU0-SMTP68A5A194041773133A3E93C9910 at phx.gbl>
> Content-Type: text/plain; charset="utf-8"
>
> As the title sais.
> Are there any mustreads, and what sources are reliable. There isnt alot on this on Wordpress.org or i missed it.
>
> or should I just learn more php?
>
>
> Skickat fr?n min HTC
>
> ----- Reply message -----
> Fr?n: "Emil Uzelac" <emil at themeid.com>
> Till: <theme-reviewers at lists.wordpress.org>
> Rubrik: [theme-reviewers] Join the team
> Datum: m?n, apr 18, 2011 04:04
> menu is OK, widgetized menu is OK too ;)----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 Sun, Apr 17, 2011 at 7:54 PM, Chip Bennett <chip at chipbennett.net> wrote:
>
>
> I actually disagree regarding the "Menu" Widget - maybe. :)
> If the Theme simply provides a Widgetized sidebar, and no "Navigation Menu" markup, then this approach is fine. However, if the Theme has what would reasonably considered to be the design intent of a Navigation Menu, then that implementation must support the core Nav Menu feature.
>
>
>
> Basically: Themes are not *required* to provide Navigation Menus, but if they *do* provide a Navigation Menu, then it must support the core functionality.
> Chip
>
>
>
>
> On Sun, Apr 17, 2011 at 7:13 PM, Emil Uzelac <emil at themeid.com> wrote:
>
>
>
> 1. The theme uses the text domain of Twenty Ten in a few spots. If I remember correctly from lurking on the list this is a reason for failure of the theme.
>
>
>
>
> Yes, Twenty Ten should be changed to i.e. silverville?2. They use a widget for the menu and instruct the user to add the Pages widget. It seems to me that it's a much better option to use a WordPress menu which defaults to list pages. I don't believe this is an issue for a fail but probably a good suggestion as an update to the user.
>
>
>
>
> This part is just fine. As long as there are options for menu and being fully functional.?
> 3. The CSS fails on the background-color since it's value is set to 'none' when it should be set to 'transparent'. Would this typically be a reason for failure if all other CSS is okay?
>
>
>
>
> background-color: none; is invalid and it should be transparent. Recommendation would be background: none; or background-image: none; and not many people use the last one anymore. It's shortcoded via background.
>
>
>
>
> ?4. There are a number of other 'recommended' items in Theme Check but these are not a reason to fail a theme correct?
>
>
>
>
> Recommended isn't required, if recommendations don't cause? problems, no need to fail.
>
>
>
>
> 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 Sun, Apr 17, 2011 at 7:01 PM, Curtis McHale <curtis at curtismchale.ca> wrote:
>
>
>
>
>
> I have a few questions about my first theme review.
> 1. The theme uses the text domain of Twenty Ten in a few spots. If I remember correctly from lurking on the list this is a reason for failure of the theme..
>
>
>
>
>
>
> 2. They use a widget for the menu and instruct the user to add the Pages widget. It seems to me that it's a much better option to use a WordPress menu which defaults to list pages. I don't believe this is an issue for a fail but probably a good suggestion as an update to the user.
>
>
>
>
>
>
> 3. The CSS fails on the background-color since it's value is set to 'none' when it should be set to 'transparent'. Would this typically be a reason for failure if all other CSS is okay?
>
>
>
>
>
>
> 4. There are a number of other 'recommended' items in Theme Check but these are not a reason to fail a theme correct?
> I could probably go further in to the theme but I wanted to double check on the text domain item before I did.
>
>
>
>
>
>
>
> Curtis McHale
> PH: 604.751.3482
> http://www.curtismchale.ca
> Linkedin: http://www.linkedin.com/in/curtismchale
>
>
>
>
>
>
>
> Twitter: twitter.com/curtismchale
>
>
>
>
> On Sat, Apr 16, 2011 at 4:39 PM, Edward Caissie <edward.caissie at gmail.com> wrote:
>
>
>
>
>
>
>
> Here ya go: http://themes.trac.wordpress.org/ticket/3320
> This will get you started ...
>
>
> Cais.
>
> On Sat, Apr 16, 2011 at 7:15 PM, Curtis McHale <curtis at curtismchale.ca> wrote:
>
>
>
>
>
>
>
>
>
> Figured I'd finally get off my butt and join the team after looking at the list for a few months. My trac name is curtismchale
>
>
>
>
>
>
>
>
>
> Curtis McHale
> PH: 604.751.3482
> http://www.curtismchale.ca
>
>
> Linkedin: http://www.linkedin.com/in/curtismchale
> Twitter: twitter.com/curtismchale
>
>
>
>
>
>
>
>
>
>
>
>
>
> _______________________________________________
>
> 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
>
>
>
>
>
> _______________________________________________
>
> 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/20110418/9da3f35c/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 56
> ***********************************************
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110418/18d14e41/attachment-0001.htm>
More information about the theme-reviewers
mailing list