WordPress.org

Make WordPress Themes

Attachments (6)

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

Download all attachments as: .zip

Change History (10)

comment:1 jtnh3212 years ago

Will review as soon as I am free.

comment:2 garinungkadol2 years ago

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

garinungkadol2 years ago

garinungkadol2 years ago

garinungkadol2 years ago

comment:3 garinungkadol2 years ago

  • 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 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 garinungkadol2 years ago

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