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

[Daniel]

Are people getting confused or just not reading?

On 10/9/13, Chip Bennett <chip at chipbennett.net> wrote:
> 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
>>
>>
>


-- 
Regards,
Daniel Fenn