[theme-reviewers] [WordPress Themes] #14750: THEME: Gold - 1.2

Chip Bennett chip at chipbennett.net
Tue Oct 8 14:02:07 UTC 2013


Guys,

Please try to keep ticket-specific comments in-ticket. That's what the
ticket comments are for. :)


On Tue, Oct 8, 2013 at 10:01 AM, Manas Usharia <manas at techacadium.com>wrote:

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


More information about the theme-reviewers mailing list