WordPress.org

Make WordPress Themes

Opened 16 months ago

Closed 15 months ago

Last modified 14 months ago

#10562 closed theme (not-approved)

THEME: Able - 1.0

Reported by: automattic Owned by: tikendramaitry
Priority: Keywords: theme-able
Cc: themes@…

Description

Able - 1.0

A multipurpose, traditional blog theme which contains up to a large three-column layout that is clean, responsive, and highly readable.

Theme URL - http://theme.wordpress.com/themes/able/
Author URL - http://automattic.com/

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

All previous tickets for this theme: http://themes.trac.wordpress.org/query?keywords=~theme-able

https://themes.svn.wordpress.org/able/1.0/screenshot.png

Change History (4)

comment:1 tikendramaitry15 months ago

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

comment:2 tikendramaitry15 months ago

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

Theme Plugin Test:
Plugin "Theme Check": Pass
Plugin "Debogger": Pass
Plugin "Log Deprecated Notices": None

Theme Unit Test:

  • Content overflowing out of wrapper box:

http://www.diigo.com/item/image/34gup/g6cr

Required Issues:

  • Inappropriate text domain found footer.php line 18
<?php _e( 'Menu', 'a-simpler-time' ); ?>
  • This Post has no Post Title, but must still display a link to the single-page Post view.

http://www.diigo.com/item/image/34gup/hnd8
http://codex.wordpress.org/Theme_Unit_Test#.28no_title.29

Theme Recommendation:

  • No reference to the_post_thumbnail() was found in the theme. It is recommended that the theme implement this functionality instead of using custom fields for thumbnails.

Review Summary:

  • Theme has required issues (see above).
  • Ticket being resolved as NOT-APPROVED.

comment:3 automattic15 months ago

This Post has no Post Title, but must still display a link to the single-page Post view.

Can you upload a clearer screenshot? All posts without titles already have permalinks available (click on the date), so I'm not sure what this required issue refers to.

Content overflowing out of wrapper box:

Would you mind uploading as an attachment the HTML that you're using in that post? I'd love to figure this one out but the tables in my demo data aren't reproducing the issue.

Thank you.

Last edited 15 months ago by automattic (previous) (diff)

comment:4 kobenland14 months ago

Having the theme not-approved for these arguably minor issues is pretty tough, especially given the amount of time it will have wait for a second review. We usually approve those kinds of tickets and mark the issues required for a future resubmission.

Note: See TracTickets for help on using tickets.