<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 class="im">#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.<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 class="HOEnZb"><div class="h5">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></blockquote></div><br></div>