Opened 8 years ago
Closed 7 years ago
#38847 closed theme (not-approved)
THEME: Masonry2017 – 1.0.0
| Reported by: |
atmoon
|
Owned by: |
bhoot
|
|---|---|---|---|
| Priority: | new theme | Keywords: | theme-masonry2017 child-theme parent-twentyseventeen accessibility-ready |
| Cc: | atmoon.geo@… |
Description
Masonry2017 - 1.0
Child Theme For the Twenty Seventeen with a masonry layout on the frontpage and archive pages based on Blogsonry.
Theme URL - http://masonry2017.wpforwhat.com
Author URL - http://wpforwhat.com
SVN - https://themes.svn.wordpress.org/masonry2017/1.0
ZIP - https://wordpress.org/themes/download/masonry2017.1.0.zip?nostats=1
Parent Theme: https://wordpress.org/themes/twentyseventeen
History:
| Ticket | Summary | Status | Resolution | Owner |
|---|---|---|---|---|
| #38847 | THEME: Masonry2017 – 1.0.0 | closed | not-approved | |
Theme Check Results:
- REQUIRED: Could not find DOCTYPE. See: https://codex.wordpress.org/HTML_to_XHTML
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"<br />"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"?>
- REQUIRED: Could not find wp_head. See: wp_head
<?php wp_head(); ?>
- REQUIRED: Could not find language_attributes. See: language_attributes
<html <?php language_attributes(); ?>
- REQUIRED: Could not find charset. There must be a charset defined in the Content-Type or the meta charset tag in the head.
- REQUIRED: Could not find add_theme_support( 'automatic-feed-links' ). See: add_theme_support
<?php add_theme_support( $feature ); ?>
- REQUIRED: Could not find comments_template. See: comments_template
<?php comments_template( $file, $separate_comments ); ?>
- REQUIRED: Could not find wp_list_comments. See: wp_list_comments
<?php wp_list_comments( $args ); ?>
- REQUIRED: Could not find comment_form. See: comment_form
<?php comment_form(); ?>
- REQUIRED: Could not find the comment-reply script enqueued. See: Migrating Plugins and Themes to 2.7/Enhanced Comment Display
<?php if ( is_singular() ) wp_enqueue_script( "comment-reply" ); ?>
- REQUIRED: The theme doesn't have comment pagination code in it. Use paginate_comments_links() or the_comments_navigation or the_comments_pagination or next_comments_link() and previous_comments_link() to add comment pagination.
- REQUIRED: No content width has been defined. Example:
if ( ! isset( $content_width ) ) $content_width = 900;
- REQUIRED: add_custom_background() found in the file functions.php. Deprecated since version 3.4. Use add_theme_support( 'custom-background', $args ) instead.
Line 91: add_custom_background();
- RECOMMENDED: 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.
- RECOMMENDED: Could not find the file readme.txt in the theme. Please see Theme_Documentation for more information.
- REQUIRED: This theme doesn't seem to support the standard avatar functions. Use get_avatar or wp_list_comments to add this support.
- RECOMMENDED: No reference to post-thumbnails was found in the theme. If the theme has a thumbnail like functionality, it should be implemented with add_theme_support( "post-thumbnails" )in the functions.php file.
- REQUIRED: Text Domain: is missing from your style.css header.
- REQUIRED: .bypostauthor css class is needed in your theme css.
- REQUIRED: .aligncenter css class is needed in your theme css.
- REQUIRED: .wp-caption-text css class is needed in your theme css.
- REQUIRED: .gallery-caption css class is needed in your theme css.
- REQUIRED: .screen-reader-text css class is needed in your theme css. See See: the Codex for an example implementation.
- WARNING: Found wrong tag, remove masonry-layout from your style.css header.
- WARNING: Found wrong tag, remove gallery from your style.css header.
- REQUIRED: This theme doesn't seem to display tags. Modify it to display tags in appropriate locations.
- Warning: More than one text-domain is being used in this theme. This means the theme will not be compatible with WordPress.org language packs. The domains found are twentyseventeen, masonry
- REQUIRED: No reference to add_theme_support( "title-tag" ) was found in the theme.
- REQUIRED: The theme must not used the <title> tags.
- REQUIRED: The theme must not call to wp_title().
- REQUIRED: The theme appears to use dynamic_sidebars() but no register_sidebar() was found. See: register_sidebar
<?php register_sidebar( $args ); ?>
Attachments (1)
Change History (10)
#1
@
8 years ago
- Keywords changed from theme-masonry2017, child-theme, parent-twentyseventeen, accessibility-ready to theme-masonry2017 child-theme parent-twentyseventeen accessibility-ready
Hi!
You need to upload your updated version here: https://wordpress.org/themes/upload/
#2
@
8 years ago
- Summary changed from THEME: Masonry2017 – 1.0 to THEME: Masonry2017 – 1.0.0
Masonry2017 - 1.0.0
Child Theme For the Twenty Seventeen with a masonry layout on the frontpage and archive pages based on Blogsonry.
Theme URL - http://masonry2017.wpforwhat.com
Author URL - http://wpforwhat.com
SVN - https://themes.svn.wordpress.org/masonry2017/1.0.0
ZIP - https://wordpress.org/themes/download/masonry2017.1.0.0.zip?nostats=1
Parent Theme: https://wordpress.org/themes/twentyseventeen
Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=masonry2017/1.0&new_path=masonry2017/1.0.0
History:
| Ticket | Summary | Status | Resolution | Owner |
|---|---|---|---|---|
| #38847 | THEME: Masonry2017 – 1.0.0 | closed | not-approved | |
Theme Check Results:
- REQUIRED: Could not find DOCTYPE. See: https://codex.wordpress.org/HTML_to_XHTML
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"<br />"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"?>
- REQUIRED: Could not find wp_head. See: wp_head
<?php wp_head(); ?>
- REQUIRED: Could not find language_attributes. See: language_attributes
<html <?php language_attributes(); ?>
- REQUIRED: Could not find charset. There must be a charset defined in the Content-Type or the meta charset tag in the head.
- REQUIRED: Could not find add_theme_support( 'automatic-feed-links' ). See: add_theme_support
<?php add_theme_support( $feature ); ?>
- REQUIRED: Could not find comments_template. See: comments_template
<?php comments_template( $file, $separate_comments ); ?>
- REQUIRED: Could not find wp_list_comments. See: wp_list_comments
<?php wp_list_comments( $args ); ?>
- REQUIRED: Could not find comment_form. See: comment_form
<?php comment_form(); ?>
- REQUIRED: Could not find the comment-reply script enqueued. See: Migrating Plugins and Themes to 2.7/Enhanced Comment Display
<?php if ( is_singular() ) wp_enqueue_script( "comment-reply" ); ?>
- REQUIRED: The theme doesn't have comment pagination code in it. Use paginate_comments_links() or the_comments_navigation or the_comments_pagination or next_comments_link() and previous_comments_link() to add comment pagination.
- REQUIRED: No content width has been defined. Example:
if ( ! isset( $content_width ) ) $content_width = 900;
- RECOMMENDED: 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.
- REQUIRED: This theme doesn't seem to support the standard avatar functions. Use get_avatar or wp_list_comments to add this support.
- RECOMMENDED: No reference to post-thumbnails was found in the theme. If the theme has a thumbnail like functionality, it should be implemented with add_theme_support( "post-thumbnails" )in the functions.php file.
- REQUIRED: .bypostauthor css class is needed in your theme css.
- REQUIRED: .aligncenter css class is needed in your theme css.
- REQUIRED: .wp-caption-text css class is needed in your theme css.
- REQUIRED: .gallery-caption css class is needed in your theme css.
- REQUIRED: .screen-reader-text css class is needed in your theme css. See See: the Codex for an example implementation.
- WARNING: Found wrong tag, remove search column from your style.css header.
- REQUIRED: This theme doesn't seem to display tags. Modify it to display tags in appropriate locations.
- REQUIRED: No reference to add_theme_support( "title-tag" ) was found in the theme.
- REQUIRED: The theme must not used the <title> tags.
- REQUIRED: The theme must not call to wp_title().
- REQUIRED: The theme appears to use dynamic_sidebars() but no register_sidebar() was found. See: register_sidebar
<?php register_sidebar( $args ); ?>
This ticket was mentioned in Slack in #themereview by mignonstyle. View the logs.
8 years ago
#5
@
7 years ago
- Owner liamdickinson deleted
I am sorry this review is taking so long. Sometimes people are unable to carry on the review, this may have happened this time. As a result, I am going to add this to the new queue again as a priority. This may mean it still takes time, but will prevent this ticket being held for so long by a reviewer that isn't able to carry on. Thanks for your patience.
If you are the reviewer and able to do this review, please carry on and request you get added back in Slack #themereview or you can take on another review when you have time again.
#7
@
7 years ago
Please fix the following errors and resubmit your theme again.
- Warning: More than one text-domain is being used in this theme. This means the theme will not be compatible with WordPress.org language packs.
- The domains found are twentyseventeen, masonry
- WARNING: Found wrong tag, remove search column from your style.css header.
- Replace all references form Twenty Seventeen to Masonry2017
Regards,
Bhoot
Note: See
TracTickets for help on using
tickets.
new revision