#4494 closed theme (not-approved)

THEME: Energy - 1.04

Reported by: bradthomas127 Owned by: garinungkadol
Priority: major Keywords: theme-energy, synchold
Cc: bradthomas127@…

Description

Energy - 1.04

The Energy theme is a stylish, neat, and fully customizable WordPress? theme. Make it yours with the easy customizable theme options with a simple "check-box" interface to easily change colors, fonts, header, menus, sidebars, and much more. Uses "Superfish" menu effects and it has built-in pagination for post pages and comments. Energy has six widget areas with an optional sidebar on the left or right hand side and has an optional one-column page template that removes the sidebar. Special styles for posts in the "Asides", "Status", and "Gallery" categories and a active support forum.

Theme URL - http://wp-ultra.com
Author URL -

SVN - http://themes.svn.wordpress.org/energy/1.04
ZIP - http://wordpress.org/extend/themes/download/energy.1.04.zip?nostats=1

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/energy/1.03&new_path=/energy/1.04

All previous tickets for this theme: http://themes.trac.wordpress.org/query?col=id&col=summary&col=keywords&col=owner&col=status&col=resolution&keywords=~theme-energy&order=priority

http://themes.svn.wordpress.org/energy/1.04/screenshot.png

Attachments (1)

menu-ie7.jpg (48.2 KB) - added by garinungkadol 22 months ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 22 months ago by garinungkadol

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

Changed 22 months ago by garinungkadol

comment:2 Changed 22 months ago by garinungkadol


THEME REVIEW RESULTS
Below are the theme review results. Anything marked FAIL needs to be addressed. Anything marked REQUIRED is needed to be fixed while anything marked RECOMMENDED is strongly advised. Other points may be raised purely for information.

Just the last bit of code quality issues to take care of and I can approve your theme. I'll leave the ticket open for the next two days. Once you've re-submitted your theme, please comment here with the link to the new ticket. If you need more time, just let me know.

RECOMMENDATION: NOT-APPROVED if code quality issues are not fixed.


Issues from Previous Review: PASS

  • FIXED: wp_head() is directly before </head>.
  • FIXED: Search form in header.php is included using get_search_form().
  • FIXED: All functions have been prefixed
  • MOSTLY-FIXED: The only issue in IE7 is that sub-menu items are hidden. See menu-ie7.jpg. Make a note this in the readme.

Theme Check Results: PASS

  • RECOMMENDED: Author URI: is missing from your style.css header.

Code Quality: FAIL

  • REQUIRED: Since you are providing translation support, all text must be translatable. Check for missing text domains in search.php, nrg-options.php. Also make use of consistent text domain names. In your options file, the text domain must be \'energy not Energy. Also do not use the 'twentyten' text domain in single.php
  • REQUIRED: All custom global variables must be prefixed in your options files: $themename, $shortname, $option_group, $option_name
  • REQUIRED: Themes are required to use checked() and selected() for checkbox and select options.

Information
If there are any issues once resolved, please re-submit your Theme using the uploader tool on wordpress.org/extend/themes, and a new Trac ticket will be generated.
Note:

  • Re-submitted Themes are given review priority over new Themes.

You can use the following tools to recreate the issues raised above if any have been:

Themes should be reviewed using define('WP_DEBUG', true); in wp-config.php
You can also view the Theme Review page.

Theme developer questions, comments, and feedback are welcome and encouraged. Please use one of the following methods:

  • 1) Leave a comment on this ticket
  • 2) Email the theme-reviewers email list
  • 3) Use the #wordpress-themes channel on irc.freenode.net

comment:3 follow-up: Changed 22 months ago by bradthomas127

@garinungkadol, Thanks for the review. Can i ask what you use to test IE7? I've been using Adobe Browserlab and its just a screen shot so i cant see the drop down menus.

comment:4 in reply to: ↑ 3 Changed 22 months ago by garinungkadol

Replying to bradthomas127:

@garinungkadol, Thanks for the review. Can i ask what you use to test IE7? I've been using Adobe Browserlab and its just a screen shot so i cant see the drop down menus.


I have Internet Explorer 7 installed on my computer :) The menus work fine in Firefox and Chrome. For me, that kind of an issue is something you can look into in a future release. Personally I don't develop with IE7 in mind anymore.

comment:5 Changed 22 months ago by bradthomas127

@garinungkado Thanks for the review, you can go ahead and close this ticket.

comment:6 Changed 22 months ago by garinungkadol

  • Keywords synchold added
  • Resolution set to not-approved
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.