WordPress.org

Make WordPress Themes

Opened 2 years ago

Closed 2 years ago

Last modified 18 months ago

#5825 closed theme (not-approved)

THEME: GroundWork - 1.0

Reported by: bhadaway Owned by: saymar90
Priority: Keywords: theme-groundwork
Cc: bhadaway@…

Description

GroundWork - 1.0

GroundWork is the groundwork already prepared for any website, mobile site, social network, blog or forum. A responsive framework to jumpstart any project.

Theme URL - http://calmestghost.com/groundwork/
Author URL - http://calmestghost.com/

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

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

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

Attachments (6)

bottom-menu.png (9.3 KB) - added by saymar90 2 years ago.
Clearing-floats-page.png (33.8 KB) - added by saymar90 2 years ago.
definition-list.png (16.5 KB) - added by saymar90 2 years ago.
Page-with-all-widgets-areas-actived.png (112.2 KB) - added by saymar90 2 years ago.
table.png (3.5 KB) - added by saymar90 2 years ago.
wide-image.png (249.4 KB) - added by saymar90 2 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 saymar902 years ago

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

saymar902 years ago

saymar902 years ago

saymar902 years ago

saymar902 years ago

comment:2 saymar902 years ago

Suggest

Not approved.

Theme Check Recommended

  • Could not find the file readme.txt in the theme. Please see Theme_Documentation for more information.
  • No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.
  • No reference to add_custom_image_header was found in the theme. It is recommended that the theme implement this functionality if using an image for the header.
  • No reference to add_custom_background() was fo

Layout

Code Quality

  • If incorporated into the Theme, custom template files are required to be called using get_template_part() or locate_template()
  • Since you use __(); or _e(); your theme support translation. You should internationalize all text in the theme. If you don't want to support translation just remove all __() or _e().

comment:3 saymar902 years ago

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

comment:4 follow-up: bhadaway2 years ago

  • Previous or Next image links don't display to image page.

I think perhaps you made a mistake on this one or I'm misunderstanding? Both Post Format Test: Gallery and Layout Test which are the surrounding posts (at least on my setup) link to it just fine.

  • If incorporated into the Theme, custom template files are required to be called using get_template_part() or locate_template()

See here for additional information: How To Load Files Within WordPress Themes
include( TEMPLATEPATH . '/file.php' ) must not be used to call Theme template part files.

  • Since you use (); or _e(); your theme support translation. You should internationalize all text in the theme. If you don't want to support translation just remove all () or _e().

Not sure what to do specifically for either of these.

Thanks, Bryan

comment:5 in reply to: ↑ 4 saymar902 years ago

Replying to bhadaway:

  • Previous or Next image links don't display to image page.

I think perhaps you made a mistake on this one or I'm misunderstanding? Both Post Format Test: Gallery and Layout Test which are the surrounding posts (at least on my setup) link to it just fine.

You are misunderstanding. When we are in Post Format Test: Gallery and you click on an image, you go to attachment page but previous and next image links should display like previous and next post links on single post page.

  • If incorporated into the Theme, custom template files are required to be called using get_template_part() or locate_template()

See here for additional information: How To Load Files Within WordPress Themes
include( TEMPLATEPATH . '/file.php' ) must not be used to call Theme template part files.

You should use get_template_directory() instead of TEMPLATEPATH

  • Since you use (); or _e(); your theme support translation. You should internationalize all text in the theme. If you don't want to support translation just remove all () or _e().

You should internationalize all text in the theme ie if you have Archives you should <?php _e( 'Archives', 'groundwork' ); ?> instead of or if you don't want to support translation just remove all _e(); and __(); functions.

comment:6 bhadaway2 years ago

You are misunderstanding. When we are in Post Format Test: Gallery and you click on an image, you go to attachment page but previous and next image links should display like previous and next post links on single post page.

Is there an example of this somewhere, both in demo and code? I've actually never seen or even heard of this WordPress feature before.

Thanks, Bryan

Note: See TracTickets for help on using tickets.