THEME: Luxury - 1.0

[JarretC]

Luxury - 1.0

Luxury is a simple blog style theme with a touch of class

Theme URL - ​http://jarretcade.net/luxury
Author URL - ​http://jarretcade.com

SVN - ​http://themes.svn.wordpress.org/luxury/1.0
ZIP - ​http://wordpress.org/extend/themes/download/luxury.1.0.zip?nostats=1

All previous tickets for this theme: http://themes.trac.wordpress.org/query?keywords=~theme-luxury

Thank you for submitting your theme.
If you have any questions about the review please reply to this post.

Suggestion: not approved.

license: Pass
links: Pass
layout: Pass
theme check: pass
theme options and functions.php: pass

Required:

Themes must not generate any HTML validation errors:
-You have a tiny error in your html code.
No p element in scope but a p end tag seen.

-this seems to be caused by comments.php line 7-8 where endif is before the closing of the p tag.

(end of requirements)

Notes and recommendations:
-Consider adding google fonts with wp_enqueue_scripts instead of using add_action('wp_head').

-You have started localizing your theme, I would strongly recommend that you finish it.

When you have corrected the above required issue please submit your theme again and reply to this post with your new ticket number. Reviewed themes are given priority in the que.

Good catch on the p tag in comments.php.

Just curious, which validator did you use? Ran the theme through ​http://validator.w3.org/ which is recommended on the ​http://codex.wordpress.org/Theme_Review page and it didn't catch it.

Ok, new ticket is at http://themes.trac.wordpress.org/ticket/8731 with required changes.

@poena since @JarretC already fixed the requirement already I will just close this ticket instead of not-approved and you're already assigned to that as well. Please let me know if that was all right :) This will speed up the approval process :)

Thanks!

yes thats allright