[theme-reviewers] Please assign me a theme to review

[Daniel Tara]

None other then the theme review guidelines:

http://codex.wordpress.org/Theme_Review

There's also this list, very good and useful, but not everything in  
there is required by the theme guidelines.

http://www.wplover.com/lab/theme-development-checklist



--- On Tue, 3/29/11, carolina n <myazalea at hotmail.com> wrote:

From: carolina n <myazalea at hotmail.com>
Subject: Re: [theme-reviewers] Please assign me a theme to review
To: theme-reviewers at lists.wordpress.org
Date: Tuesday, March 29, 2011, 6:48 PM





  Is there a complete checklist for theme reviewers, i liked how  
Michael (mfields) did his.

> Date: Tue, 29 Mar 2011 17:34:32 +0300
> From: contact at onedesigns.com
> To: theme-reviewers at lists.wordpress.org
> Subject: Re: [theme-reviewers] Please assign me a theme to review
>
> Hi Carolina,
>
> Yes, you just reviewed an Automattic theme. How cool is that? *grin*
>
> Here's the official blog post to it:
> http://en.blog.wordpress.com/2008/09/12/depo-masthead-theme-launch/
>
> Here's some feedback and suggestions;
> Apparently the theme intends to create a custom pagination, to show   
> only the first 3 posts on the front page and a custom archive-based   
> navigation. This could be a potential issue since it may require   
> additional tweaking, Apparently this theme would run smoother if the  
>  front page would be set to a static page. Otherwise, I would think  
> the  use of the template front-page.php in this case would have been  
> a  better choice instead of home.php. With the proper documentation  
> such  a structure may be fine, but still subjective and the final  
> decision  may be set to debate. I personally would not accept this  
> because it  appears there is no main post navigation if the front  
> page is set to  latest posts.
>
> The WP admin bar does not break for me. What version of Firefox do  
> you  use, could you include a screenshot?
>
> In the future please also copy the errors/notices you receive from   
> WP_DEBUG and also the Theme Check requirements and recommendations.
>
> This theme also has a theme options page. In the future please also  
> do  a security review for such themes, checking if attributes are   
> correctly escaped, there are no XSS vulnerabilities, nonce fields  
> are  added. If you are not familiar with the Settings/options API  
> then  leave a request for someone else to do it.
>
> This theme passes security tests, the only recommendation would be  
> to  use the Settings API instead of the Options API.
>
> Here are some other things you didn't catch:
> The theme uses inconsistent function prefixes: sometimes functions   
> start with depo_ sometimes with depomasthead_
> Also, functions like previous_archive_link and next_archive_link are  
>  not prefixed at all and have very common names.
>
> If other reviewers have other thoughts please share them, I only had  
> a  quick look over the theme.
>
> Daniel
>
>
>
> --- On Tue, 3/29/11, carolina n <myazalea at hotmail.com> wrote:
>
> From: carolina n <myazalea at hotmail.com>
> Subject: Re: [theme-reviewers] Please assign me a theme to review
> To: theme-reviewers at lists.wordpress.org
> Date: Tuesday, March 29, 2011, 5:06 PM
>
>
>
>
>
>
> Done
>
> -wasn't sure if this was some kind of trick or error, or did i just   
> fail an automattic theme ;)
>
>
>
>
>
>
>
>
> > Date: Tue, 29 Mar 2011 15:37:48 +0300
> > From: contact at onedesigns.com
> > To: theme-reviewers at lists.wordpress.org
> > Subject: Re: [theme-reviewers] Please assign me a theme to review
> >
> > I'm sorry, wrong pick. I picked the most recent one instead of the oldest.
> >
> > Please review this one:
> >
> > https://themes.trac.wordpress.org/ticket/3085
> >
> > My apologies.
> >
> > Daniel
> >
> > --- On Tue, 3/29/11, Daniel Tara <contact at onedesigns.com> wrote:
> >
> > [Hide Quoted Text]
> > From: Daniel Tara <contact at onedesigns.com>
> > Subject: Re: [theme-reviewers] Please assign me a theme to review
> > To: theme-reviewers at lists.wordpress.org
> > Date: Tuesday, March 29, 2011, 2:49 PM
> > Here you go:
> >
> > https://themes.trac.wordpress.org/ticket/3384
> >
> > Daniel
> >
> > --- On Tue, 3/29/11, carolina n <myazalea at hotmail.com>
> > wrote:
> >
> > From: carolina n <myazalea at hotmail.com>
> > Subject: [theme-reviewers] Please assign me a theme to
> > review
> > To: theme-reviewers at lists.wordpress.org
> > Date: Tuesday, March 29, 2011, 12:39 PM
> >
> > Please assign me a theme or two to review,
> >
> > my username is poena.
> >
> >
> > -----Inline Attachment Follows-----
> >
> > _______________________________________________
> > 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
>
>
> -----Inline Attachment Follows-----
>
> _______________________________________________
> 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


-----Inline Attachment Follows-----

_______________________________________________
theme-reviewers mailing list
theme-reviewers at lists.wordpress.org
http://lists.wordpress.org/mailman/listinfo/theme-reviewers