[theme-reviewers] Shades 1.5 review

[Tom Lany]

I put this up on Trac.  Sign up for an account on the forums, and you 
can post reviews directly on the ticket.

Tom Lany
http://tomlany.net

On 6/19/10 5:11 PM, Simon Prosser wrote:
> This is my review for shades 1.5.
>
> There is already a 1.4 version in the repo, this seems like a refresh 
> for wordpress 3.0, Ive highlighted the minor errors
> I'm sure the author will fix the css error.
>
>
>    * The theme uses valid HTML.
>
> *Seems that the search form declares ID "searchform" twice, needs to 
> use class instead of id for one of those divs.
> *
>    * The theme uses valid CSS.
>
> *just one error ( if you ignore the css3 moz errors )
> #header-center h2 shows colour #00000 should be #000000*
>
>    * The theme uses minimal CSS hacks.
>
> *Theme has an ie6 file, but its empty.*
>
>    * When referencing other files within the same theme, avoid 
> hard-coded URIs and file paths. Instead reference the URIs and file 
> paths with bloginfo(): see Referencing Files From a Template.
>
> *Uses TEMPLATEPATH to fetch locales*
>
>    * The navigation is easy to understand.
>
> *Yes its easy to follow, but the theme has been updated for 3.0 but 
> does not support the new menu system.*
>
> * Does the theme have pages tabs in the header? If so, do they 
> overflow or have problems with child pages? Only shows level 1 pages.
>
> o To output date format: the_time( get_option( 'date_format' ) )
>
> *theme uses <?php the_time( 'M j, Y' ) ?>*
>
>  * Do embedded videos look correct?
>
> *No support links just show as shortcodes
> *
> Language support YES
>
> *Theme has language support BR included in archive*
>
>
> Obviously a pass as the author already has 1.4 passed.
>
>
> -- 
> My Blog: http://www.pross.org.uk/
> Plugins : http://www.pross.org.uk/plugins/
> Themes: http://wordpress.org/extend/themes/profile/pross
>
>
> _______________________________________________
> 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/mailman/private/theme-reviewers/attachments/20100619/8994c042/attachment-0001.htm>