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

David Alllen david_allen1 at hotmail.com
Wed Mar 30 05:26:22 UTC 2011


Hi,
 
Please can you tll me the address of the forum, i wanted to ask a question about the best site to use which matches my requirements but somehow ended up on this email listing.
 
Also can you please remove me from this listing.
 
Thank you.
 
David. 



 


From: myazalea at hotmail.com
To: theme-reviewers at lists.wordpress.org
Date: Wed, 30 Mar 2011 07:20:25 +0200
Subject: Re: [theme-reviewers] Please assign me a theme to review




forgot;
Firefox/3.6.16
screenshot:
http://img12.imageshack.us/f/depomasthead.png/
 

 
> 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

_______________________________________________ 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/20110330/34552b23/attachment.htm>


More information about the theme-reviewers mailing list