Make WordPress Themes

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#5156 closed theme (not-approved)

THEME: webmagazine - 2.0

Reported by: webmag Owned by: garinungkadol
Priority: Keywords: theme-webmagazine
Cc: wtmmanage@…


webmagazine - 2.0

A Wordpress magazine theme based on the default Twentyten theme.Frontpage images are generated using the post thumbnail function.

Theme URL - http://wpblogmagazine.com/
Author URL - http:http://twitter.com/#!/hupseymedia

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

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/webmagazine/1.9&new_path=/webmagazine/2.0

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-webmagazine&order=priority


Attachments (5)

clearing-floats.jpg (43.8 KB) - added by garinungkadol 7 years ago.
default-header.jpg (25.8 KB) - added by garinungkadol 7 years ago.
footer.jpg (60.9 KB) - added by garinungkadol 7 years ago.
image-continue-reading.jpg (74.7 KB) - added by garinungkadol 7 years ago.
linked-image.jpg (128.7 KB) - added by garinungkadol 7 years ago.

Download all attachments as: .zip

Change History (8)

#1 @garinungkadol
7 years ago

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

7 years ago

#2 @garinungkadol
7 years ago

  • Keywords synchold added
  • Resolution set to not-approved
  • Status changed from assigned to closed

This is a complete review. 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.


Issues from Previous Review: PASS

  • FIXED: Domain name has been changed and no longer uses 'WordPress'.

Theme Information: FAIL

  • Author URI is not valid. It must be either the author's personal site or project page. Your twitter account is neither.
  • Tags: Please remove "theme-options" from list of tags in style.css as you do not have theme options.
  • Credit Link: The template homefooter.php displays two credit links. You are only allowed one link which must be either the Theme URI or Author URI.

Theme Check Results: PASS

Debugger results: PASS

Test data results: FAIL

Blog Post Index View

  • REQUIRED: Large images should not be cut off. Captions should not be cut off either. See linked-image.jpg
  • REQUIRED: Posts without titles must still display the permalink to the single post view.

Single Post View

  • Tags are not being displayed on the single post view. Is this intentional?

Visual issues / functionality / browser checks: FAIL

  • FAIL: Please check your implementation of headers (Appearance >> Header).
    • The list of default header images show up as broken images. See default-header.jpg
    • The option to display text is not available.
    • An uploaded image is not displayed on the site.

  • REQUIRED: Provide a background color for the post content area. If a background image is set, post text may not be visible.
  • RECOMMENDED: Theme should support multi-level menu links. At present theme only supports 1 level of links. You should note this in the readme.txt
  • In 1024 x 768, the second to fourth widget areas appear below the first footer widget area. See footer.jpg

Validation: PASS



Code Quality: FAIL

  • REQUIRED: wp_footer() must be placed directly before </body>
  • REQUIRED: Additional scripts should not be hard-coded in footer.php. They should be properly enqueued in functions.php.
    function mythemeprefix_scripts(){
    	if( !is_admin() ) 
    		wp_enqueue_script( 'mythemeprefix_plusone', 'http://apis.google.com/js/plusone.js', array(), false, true );
    add_action( 'wp_enqueue_scripts', 'mythemeprefix_scripts' );
  • REQUIRED: Please change all references to Twenty Ten in inline documentation in functions.php, homefooter.php, loop.php
  • REQUIRED: Template files must be included using either get_template_part() or locate_template(). See Line 34 of home.php

This following should be of assistance in getting your theme to meet all the required criteria:

Once the points above have been addressed appropriately using the available information found at the links above; please thoroughly review your theme and re-submit it using the uploader tool: http://wordpress.org/extend/themes/upload/
This will generate a new Trac ticket.

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

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

#3 @chipbennett
7 years ago

  • Keywords synchold removed
Note: See TracTickets for help on using tickets.