[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