WordPress.org

Make WordPress Themes

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#589 closed theme (not-approved)

THEME: Suffusion - 3.6.0

Reported by: themetracbot Owned by: cais
Priority: Keywords: theme-suffusion,
Cc:

Description

Suffusion - 3.6.0

An elegant, versatile and browser-safe theme with a power-packed set of options. It has 19 widget areas, one-column, two-column and three-column fixed-width formats, 10 pre-defined templates, 17 pre-defined color schemes, two customizable multi-level drop-down menus, featured posts, a magazine layout, tabbed sidebars, widgets for Twitter, Social Networks and Google Translator, translations in many languages and RTL language support. WP 3.0 Menus, Custom Post Types and Custom Taxonomies are integrated. A BuddyPress support pack is available on the theme page. Support forum at http://www.aquoid.com/forum.

Theme URL - http://www.aquoid.com/news/themes/suffusion/
Author URL - http://mynethome.net/blog

SVN - http://themes.svn.wordpress.org/suffusion/3.6.0

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

Change History (5)

#1 @sayontan
6 years ago

Submitting this version based on some feedback I have seen for other themes. Note that I have been on the approval queue for 3 weeks now. Can this please be approved without being pushed to the bottom of the queue again?

Here is the change log w.r.t. version 3.5.9:

  1. Added two new sidebars - Wide Sidebar (Top) and Wide Sidebar (Bottom)
  2. Fixed a bug in the "No Sidebars" template where the maximum width of the image was being set incorrectly.
  3. Fixed a bug in the XHTML validation in case gradient images are used.
  4. Switched the JavaScript files to be loaded in the footer by default, in Blog Features -> Site Optimization
  5. Added capability to display text in multiple columns using short codes [suffusion-multic] and [suffusion-column]
  6. Added translation support for child themes.
  7. Changed behavior to exclude posts from the static Featured Content section in the rest of the content on a page.
  8. Made minor style tweaks to Minima
  9. Replaced occurrences of include(TEMPLATEPATH . '/searchform.php') with get_search_form()
  10. Replaced occurrences of file includes with get_template_part()
  11. Fixed XHTML validation errors in GZip compression mode.
  12. Removed support for comments from WP versions older than 2.7. Suffusion anyway has been relying on at least 2.8 for widgets since a long time.
  13. Added call to new function comment_form() in comments.php.
  14. Moved licensing information to style.css file.
  15. Added call to wp_login_form() in login.php.
  16. Updated Brazilian Portuguese translation.

#2 @cais
6 years ago

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

#3 @cais
6 years ago

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

Theme Review:
=> Themes should be reviewed using "define('WP_DEBUG', true);" in wp-config.php
=> Themes should be reviewed using the test data from the Theme Checklists (TC)

--
WP_DEBUG et al.:

Other warnings:

  • Debug: Undefined index: wsidebar on line 644 of /wp-content/themes/suffusion/custom-styles.php
  • Debug: Undefined index: wsidebar on line 648 of /wp-content/themes/suffusion/custom-styles.php

--
TC:
=> http://codex.wordpress.org/Theme_Unit_Test#Page_With_Comments_Disabled

--
General Observations:

  • Nicely implemented. I will be looking forward to seeing these last items addressed.

--
Overall: not-accepted

  • Items marked (=>) must be addressed.
  • Other items noted should be addressed and corrected as needed.
  • Additional review may be required once the above issues are resolved.

#4 @sayontan
6 years ago

I checked with the checklist and it says:

Page With Comments Disabled

  • Comments should not be displayed.
  • No "Comments Disabled" message should be displayed.
  • Layout not adversely impacted by minimal page content.

I believe the reason for non-acceptance was #2 above (No "Comments Disabled" message should be displayed). There is an option in the theme to enable/disable this text. You can go to Blog Features -> Comment Settings and set where you want or don't want the message.

I would like to request a reconsideration of the theme for approval if this is acceptable.

#5 @sayontan
6 years ago

I have addressed the reported issues: http://themes.trac.wordpress.org/ticket/598

Note: See TracTickets for help on using tickets.