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

[Chip Bennett]

The email notification you received includes the following:

Ticket URL: <http://themes.trac.wordpress.org/ticket/14750#comment:2>


This is a link to the Trac ticket. If you follow that link, you can post
comments to the ticket itself.


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

> Hi,
>
> I am confused on this ticket thing. This is my first theme for
> wordpress.org. 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.
>
> Thanks
>
>
>
> On Tue, Oct 8, 2013 at 7:35 PM, Daniel <danielx386 at gmail.com> wrote:
>
>> 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
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>
>
> _______________________________________________
> 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/20131009/c3e85260/attachment.html>