<div dir="ltr">Guys,<div><br></div><div>Please try to keep ticket-specific comments in-ticket. That's what the ticket comments are for. :)</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Oct 8, 2013 at 10:01 AM, Manas Usharia <span dir="ltr"><<a href="mailto:manas@techacadium.com" target="_blank">manas@techacadium.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Thanks Rohit,<br><br></div>I am making the changes and will resubmit only after proper reviewing. Thanks a lot for your feedback<br>
<br></div>Manas<br><div><div><div><br></div></div></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Oct 8, 2013 at 7:16 PM, WordPress Themes <span dir="ltr"><<a href="mailto:noreply@wordpress.org" target="_blank">noreply@wordpress.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>#14750: THEME: Gold - 1.2<br>
</div>--------------------------------+---------------------------<br>
Reporter: usharia | Owner: rohitink<br>
Type: theme | Status: closed<br>
Priority: previously reviewed | Resolution: not-approved<br>
Keywords: theme-gold |<br>
--------------------------------+---------------------------<br>
Changes (by rohitink):<br>
<br>
* status: reviewing => closed<br>
* resolution: => not-approved<br>
<br>
<br>
Comment:<br>
<br>
1. All styles/stylesheets must be enqueued. They are not supposed to be<br>
present in header.php file. Use {{{ wp_enqueue_script(); }}}<br>
<br>
2. Put some fallback code in index.php file. It should not contain just<br>
header of footer.<br>
<br>
3. The wp_head() template tag must placed immediately before the closing<br>
HTML head tag.<br>
<br>
4. No scripts or script file links should be hard-coded into the footer.<br>
(These must be enqueued and hooked in appropriately.)<br>
<br>
5. The wp_footer() template tag is placed immediately before the closing<br>
HTML body tag.<br>
<br>
6. Still no license information is present in readme.txt. You need to give<br>
license details about all images/frameworks/icons/fonts used in your<br>
theme.<br>
<br>
7. Setting the front page to display latest posts, doesn't work. Please<br>
fix this issue as well. You can have your page with banner/callouts/etc as<br>
a homepage when user selects a static page. But, the latest posts should<br>
be displayed on homepage somehow with or without the featuer banner<br>
section.<br>
<br>
8. Comment Description overflows container. The site has text overflow<br>
problem at many places including post titles and content(when containing<br>
extra long words). Same problem in comment content too..<br>
<br>
9. Setting a Page to display latest posts, also doesn't work. Default<br>
functionality of wordpress can not be overridden like this.<br>
<br>
10. Credit link in the footer does not work.<br>
<br>
11. Please check your theme through theme_unit_tests. It fails a lot of<br>
tests. <a href="http://codex.wordpress.org/Theme_Unit_Test" target="_blank">http://codex.wordpress.org/Theme_Unit_Test</a><br>
<br>
Make sure you fix all these issues, and issues from previous ticket before<br>
re-submitting the theme. If you need any help, let me know.<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
Ticket URL: <<a href="http://themes.trac.wordpress.org/ticket/14750#comment:2" target="_blank">http://themes.trac.wordpress.org/ticket/14750#comment:2</a>><br>
<div><div>WordPress Themes <<a href="http://themes.trac.wordpress.org/" target="_blank">http://themes.trac.wordpress.org/</a>><br>
WordPress.org Theme Directory Reviews<br>
</div></div></font></span></blockquote></div><br></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>