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

[carolina n]

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
 		 	   		  
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110329/ad34b723/attachment.htm>