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

Manas Usharia manas at techacadium.com
Wed Oct 9 14:01:00 UTC 2013


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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20131009/175b5541/attachment.html>


More information about the theme-reviewers mailing list