THEME: KNR Decorous - 0.4

[k_nitin_r]

KNR Decorous - 0.4

An elegant theme with extensive use of white

Theme URL - ​http://www.n4express.com/blog/?page_id=853
Author URL - ​http://www.nitinkatkam.com/

SVN - ​http://themes.svn.wordpress.org/knr-decorous/0.4
ZIP - ​http://wordpress.org/extend/themes/download/knr-decorous.0.4.zip?nostats=1

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/knr-decorous/0.3&new_path=/knr-decorous/0.4

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-knr-decorous&order=id

THEME REVIEW RESULTS

RECOMMENDATION: NOT-APPROVED
The pending issues below are minor ones. Once you have made the required revisions we will be able to approve your theme.

Pending Issues

• NOT-FIXED: Floated elements must be properly cleared.

• PARTIALLY-FIXED: Registered sidebars are hooked into widgets_init though this is not an appropriate hook for knrdecorous_register_thememenu can be hooked into init while knrdecorous_register_themecapabilities is hooked into after_theme_setup.

• REQUIRED: Screenshot must fall within the prescribed dimension range of 300x225px to 320x240px.

Fixed Issues

• FIXED: References to twenty eleven have been removed.
• FIXED: Commented out code in header.php has been removed.
• FIXED: Using get_stylesheet_uri over bloginfo('stylesheet_url').
• FIXED: Using echo home_url() instead of bloginfo(url) in header.php
• FIXED: Added 'Theme URI' to style.css header
• FIXED: Wide images no longer exceed the content area.

• FIXED: Added padding between columns to separate text.
• FIXED: Each post in a column has the same post height.
• FIXED: Caption in the "Post Format Test: Image (Linked)" post is no longer cut-off
• FIXED: Posts with no Post Title displays the link to the single post page view.
• FIXED: Sub-menu links are visible when hovering over parent link.
• FIXED: Pages with comments disabled no longer shows "Comments are closed".
• FIXED: 404 (Page Not Found) displays a message instead of blank content area. I would recommend including get_search_form() to display the search form.

Hi!

Thank you for your feedback.

Regarding the hook being used in functions.php, please take a look at the following:

-=[ Scenario 1 ('init' hook) ]=-

This is the line in functions.php:
add_action('init', 'knrdecorous_register_themecapabilities');

WordPress? calls the function 'knrdecorous_register_themecapabilities', which contains the statement 'add_theme_support('automatic-feed-links');' and outputs the following in the markup of the site (right-click on the page, select "View Page Source", and look at the HEAD tag, a few lines below the TITLE tag):
<link rel="alternate" type="application/rss+xml" title="WP Test Bench &raquo; Feed" href="​http://localhost:801/wordpress/?feed=rss2" />
<link rel="alternate" type="application/rss+xml" title="WP Test Bench &raquo; Comments Feed" href="​http://localhost:801/wordpress/?feed=comments-rss2" />

-=[ Scenario 2 ('after_theme_setup' hook) ]=-

This is the line in functions.php
add_action('after_theme_setup', 'knrdecorous_register_themecapabilities');

When using the 'after_theme_setup' hook, the output is not produced.

For the other issues, I have created a new screenshot image that is of "320x240px" dimensions and I cleared the float that remains after the posts (in version 0.4, only the float used within the post Eg. the image alignLeft was cleared).

I'll post the new version shortly.

I have uploaded a new version of the theme at http://themes.trac.wordpress.org/ticket/6609