<div dir="ltr">go to <a href="http://themes.trac.wordpress.org/ticket/14750#comment:2">http://themes.trac.wordpress.org/ticket/14750#comment:2</a> and post your replies there.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Wed, Oct 9, 2013 at 7:31 PM, 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>Hi,<br><br></div>I am confused on this ticket thing. This is my first theme for <a href="http://wordpress.org" target="_blank">wordpress.org</a>. I am replying to the same mail which i am getting the feedback. Am i doing it wrong? I mean I want to discuss the feedback I got from Rohitink but not sure where and how.<br>
<br></div>Thanks<div><div class="h5"><br><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Oct 8, 2013 at 7:35 PM, Daniel <span dir="ltr"><<a href="mailto:danielx386@gmail.com" target="_blank">danielx386@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Are people getting confused or just not reading?<br>
<div><div><br>
On 10/9/13, Chip Bennett <<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>> wrote:<br>
> Guys,<br>
><br>
> Please try to keep ticket-specific comments in-ticket. That's what the<br>
> ticket comments are for. :)<br>
><br>
><br>
> On Tue, Oct 8, 2013 at 10:01 AM, Manas Usharia<br>
> <<a href="mailto:manas@techacadium.com" target="_blank">manas@techacadium.com</a>>wrote:<br>
><br>
>> Thanks Rohit,<br>
>><br>
>> I am making the changes and will resubmit only after proper reviewing.<br>
>> Thanks a lot for your feedback<br>
>><br>
>> Manas<br>
>><br>
>><br>
>><br>
>> On Tue, Oct 8, 2013 at 7:16 PM, WordPress Themes<br>
>> <<a href="mailto:noreply@wordpress.org" target="_blank">noreply@wordpress.org</a>>wrote:<br>
>><br>
>>> #14750: THEME: Gold - 1.2<br>
>>> --------------------------------+---------------------------<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<br>
>>> closing<br>
>>> HTML head tag.<br>
>>><br>
>>> 4. No scripts or script file links should be hard-coded into the<br>
>>> footer.<br>
>>> (These must be enqueued and hooked in appropriately.)<br>
>>><br>
>>> 5. The wp_footer() template tag is placed immediately before the<br>
>>> closing<br>
>>> HTML body tag.<br>
>>><br>
>>> 6. Still no license information is present in readme.txt. You need to<br>
>>> 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<br>
>>> as<br>
>>> a homepage when user selects a static page. But, the latest posts<br>
>>> 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<br>
>>> 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<br>
>>> 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>
>>> WordPress Themes <<a href="http://themes.trac.wordpress.org/" target="_blank">http://themes.trac.wordpress.org/</a>><br>
>>> WordPress.org Theme Directory Reviews<br>
>>><br>
>><br>
>><br>
>> _______________________________________________<br>
>> theme-reviewers mailing list<br>
>> <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
>><br>
><br>
<br>
<br>
</div></div><span><font color="#888888">--<br>
Regards,<br>
Daniel Fenn<br>
</font></span><div><div>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
</div></div></blockquote></div><br></div></div></div></div></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>