WordPress.org

Make WordPress Themes

Opened 2 years ago

Closed 2 years ago

#7480 closed theme (closed-newer-version-uploaded)

THEME: Fanwood - 0.1.6.2

Reported by: tungdo Owned by: kobenland
Priority: Keywords: theme-fanwood, buddypress
Cc: ttsondo@…

Description

Fanwood - 0.1.6.2

All-purpose, responsive (mobile friendly) theme with 12 optional layouts, per post layouts, multiple drop down menus, 18 widget areas, support for all 10 post formats, mobile compatible menus, and integrations for bbPress, BuddyPress, Jigoshop, and Hybrid Tabs.

Theme URL - http://devpress.com/themes/fanwood
Author URL - http://devpress.com

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

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/fanwood/0.1.6.1&new_path=/fanwood/0.1.6.2

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

https://themes.svn.wordpress.org/fanwood/0.1.6.2/screenshot.png

Change History (4)

comment:1 kobenland2 years ago

  • Owner set to kobenland
  • Status changed from new to accepted

comment:2 kobenland2 years ago

  • Keywords buddypress added

THEME REVIEW RESULTS

Code Quality

  • Themes are REQUIRED to use get_template_directory()/get_stylesheet_directory() instead of TEMPLATEPATH/STYLESHEETPATH
  • Themes are REQUIRED to use a string, rather than a variable, constant, or concatenation, as the textdomain in translation functions.
  • Themes are REQUIRED to use a unique slug as a prefix for anything in the public namespace, including all custom function names, classes, hooks, public/global variables, database entries (Theme options, post custom metadata, etc.)
  • Themes are REQUIRED to state the copyright and license information for any bundled resources not covered by the Theme's license statement. Themes are recommended to state this information in the Theme's README documentation. Are bundled fonts GPLv2?


Review Summary

  • This was not a complete review.
  • Theme has required issues (see above)


Hi Tung,

I probably should have read the description of your Theme in more detail, as I'm not sure how BuddyPress Themes are handled by the WPTRT. :)
I'll add the BuddyPress keyword and ping the mailing list to learn more about that. Please bear with me, maybe you wanna tackle the few things that I managed to find, until then.

Thank you for your contribution!
Konstantin

comment:3 mercime2 years ago

Will be reviewing the BuddyPress and bbPress portions of the theme. Mercime

comment:4 kobenland2 years ago

  • Resolution set to closed-newer-version-uploaded
  • Status changed from accepted to closed

See #7495

Note: See TracTickets for help on using tickets.