WordPress.org

Make WordPress Themes

Opened 7 years ago

Closed 7 years ago

Last modified 10 months ago

#841 closed theme (not-approved)

THEME: House in the sun travel theme - 1.1

Reported by: themetracbot Owned by: Frumph
Priority: Keywords: theme-house-in-the-sun-travel-theme,
Cc:

Description

Change History (3)

#1 @Frumph
7 years ago

  • Owner set to Frumph
  • Status changed from new to assigned

#2 @Frumph
7 years ago

  • Resolution set to not-accepted
  • Status changed from assigned to closed

=> Footer transposes to the upper right corner of the screen, not at the bottom
=> tremendous amount of CSS display issues.
=> link to author's site is a link to something that is not allowed, I realize it's your site but it's not the site about the author in particular, no spam links allowed. Remove the author's link if you do not have one.
=> your using comment_form() AND writing out the forms for comments as well, please just use comment_form()
=> need to use add_theme_support('automatic-feed-links') and
=> you have two wp_footer(); lines
=> use get_search_form() instead of including the searchform.php

/ theme URI should be the location of where the theme has a webpage for, i'm assuming your going to change it to the location in the repository when this gets accepted but that would just denote the same location people see the link on, suggest omitting it from the style.css or putting a page about the theme on authors site and linking it to that.
/ suggest using unique prefix's on functions so they don't clash with other plugins/core functions.

Items marked with => are reasons for not being approved
items marked with / are just suggestions
Items marked with - are notes

This ticket was mentioned in Slack in #themereview by jrf. View the logs.


10 months ago

Note: See TracTickets for help on using tickets.