Attachments (6)

calendar-widget.jpg (12.1 KB) - added by garinungkadol 15 months ago.
comment-reply.jpg (42.7 KB) - added by garinungkadol 15 months ago.
image-border.jpg (69.6 KB) - added by garinungkadol 15 months ago.
no-search-results.jpg (14.5 KB) - added by garinungkadol 15 months ago.
password-protected.jpg (7.4 KB) - added by garinungkadol 15 months ago.
wide-image.jpg (33.6 KB) - added by garinungkadol 15 months ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 15 months ago by jtnh321

Will review as soon as I am free.

comment:2 Changed 15 months ago by garinungkadol

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

Changed 15 months ago by garinungkadol

Changed 15 months ago by garinungkadol

Changed 15 months ago by garinungkadol

Changed 15 months ago by garinungkadol

Changed 15 months ago by garinungkadol

Changed 15 months ago by garinungkadol

comment:3 Changed 15 months ago by garinungkadol

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

THEME REVIEW RESULTS

This is a complete review.

RECOMMENDATION: NOT-APPROVED

THEME INFORMATION

  • Screenshot: The theme's screenshot must follow the prescribe dimensions of 300x225px to a maximum of 320x240px.

DEBUGGER ERRORS

On 404 Page
-- Debug: Trying to get property of non-object on line 9 of C:\xampp\htdocs\wordpress\themereview\wp-content\themes\bubbledream\header-single.php
-- Debug: Trying to get property of non-object on line 11 of C:\xampp\htdocs\wordpress\themereview\wp-content\themes\bubbledream\header-single.php

THEME UNIT TESTS

Blog Index View

  • REQUIRED: The comment text should be 'Leave a Response' not 'Leave a Respond'

Single Post View

  • RECOMMENDED: Add more styling to table tags so that tabular data fits the content area.
  • REQUIRED: Wide images that have not been re-sized in the editor are cut-off. They should be re-sized to fit the content area. wide-image.jpg
  • REQUIRED: Check the right border for large captioned images. See image-border.jpg
  • REQUIRED: Check the styling for the bottom part of the comment form when the reply button is clicked. See comment-reply.jpg

No Search Results

  • REQUIRED: Check the styling of the search form. See no-search-results.jpg
  • RECOMMENDED: You can provide archive or category links that would be helpful in locating data.

FUNCTIONALITY

  • REQUIRED: The inclusion of <meta> content for SEO functionality (robots and description) should be opt-in. The user should have the capability to enable this feature. Ideally, these tags should be placed in a function that hooks into wp_head.

CODE QUALITY

  • While localization is an optional feature, if you do intend to support it, localization must be fully implemented.
    • REQUIRED: You must declare the textdomain to be used:
        load_theme_textdomain( 'theme-slug', TEMPLATEPATH . '/languages' );
      
      	$locale = get_locale();
      	$locale_file = TEMPLATEPATH . "/languages/$locale.php";
      	if ( is_readable( $locale_file ) )
      		require_once( $locale_file );
      
    • REQUIRED: The textdomain must be the theme-slug, 'bubbledream' not 'BubbleDream?'.
    • REQUIRED: All translatable text must include a translation function.
    • REQUIRED: All translation functions must be assigned a textdomain.
  • REQUIRED: The functions add_theme_support('post-thumbnails'), add_theme_support('automatic-feed-links') and register_nav_menus() should be placed in a function that hooks into after_setup_theme.
  • REQUIRED: BubbleDream_init_method() should be hooked into wp_enqueue_scripts not init.
  • REQUIRED: You should not be deregistering jQuery. Make use of jQuery that is bundled with wordPress to avoid conflicts.
  • REQUIRED: All custom functions must be prefixed with the theme-slug. See my_remove_recent_comments_style, zxdRoboCheck
  • REQUIRED: Remove the link rel, 'theme author' in header-single.php. This is considered SEO-seeded.
  • REQUIRED: Scripts should not be enqueued in template files. Enqueue the 'comment-reply' script in a function that hooks into wp_enqueue_scripts.
  • REQUIRED: wp_footer() must be placed directly before </body>.
  • REQUIRED: Instead of including the the content of html-CFInstall.php as a template, place it in a function that hooks into wp_footer.
  • REQUIRED: The search form should be displayed using get_search_form() not get_template_part('searchform').

comment:4 Changed 14 months ago by garinungkadol

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