<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META content="text/html; charset=utf-8" http-equiv=Content-Type>
<META name=GENERATOR content="MSHTML 8.00.6001.18928">
<STYLE></STYLE>
</HEAD>
<BODY bgColor=#ffffff>
<DIV><FONT size=2 face=Arial>1) If the screenshot.png does not look like what
the end user see's when activating the theme, the screenshot.png is
invalid.</FONT></DIV>
<DIV><FONT size=2 face=Arial></FONT> </DIV>
<DIV><FONT size=2 face=Arial>2) However if you USE wp_nav_menu(); then you MUST
register it., you use it, register it.</FONT></DIV>
<DIV><FONT size=2 face=Arial></FONT> </DIV>
<DIV><FONT size=2 face=Arial>3) Line 288 in actions.php is not wrapped around
conditional <!-- if statements which should be used as an
enqueue</FONT></DIV>
<DIV><FONT size=2 face=Arial></FONT> </DIV>
<DIV><FONT size=2 face=Arial>4) We're not your theme testers, we're reviewers,
test your theme before submitting.</FONT></DIV>
<DIV><FONT size=2 face=Arial></FONT> </DIV>
<DIV><FONT size=2 face=Arial>- Phil</FONT></DIV>
<BLOCKQUOTE
style="BORDER-LEFT: #000000 2px solid; PADDING-LEFT: 5px; PADDING-RIGHT: 0px; MARGIN-LEFT: 5px; MARGIN-RIGHT: 0px">
<DIV style="FONT: 10pt arial">----- Original Message ----- </DIV>
<DIV
style="FONT: 10pt arial; BACKGROUND: #e4e4e4; font-color: black"><B>From:</B>
<A title=sayontan@gmail.com href="mailto:sayontan@gmail.com">Sayontan
Sinha</A> </DIV>
<DIV style="FONT: 10pt arial"><B>To:</B> <A
title=theme-reviewers@lists.wordpress.org
href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</A>
</DIV>
<DIV style="FONT: 10pt arial"><B>Sent:</B> Monday, August 09, 2010 9:54
AM</DIV>
<DIV style="FONT: 10pt arial"><B>Subject:</B> Re: [theme-reviewers] [WordPress
Themes] Suffusion,new version 3.6.2</DIV>
<DIV><BR></DIV>Hello,<BR>I have been following the feedback posted for this
version of the theme and I believe the review is inaccurate and not thorough.
Here is what was posted:<BR>
<BLOCKQUOTE
style="BORDER-LEFT: rgb(204,204,204) 1px solid; MARGIN: 0pt 0pt 0pt 0.8ex; PADDING-LEFT: 1ex"
class=gmail_quote>
<OL>
<LI><SPAN
style="WIDOWS: 2; TEXT-TRANSFORM: none; TEXT-INDENT: 0px; BORDER-COLLAPSE: separate; FONT: medium 'Times New Roman'; WHITE-SPACE: normal; ORPHANS: 2; LETTER-SPACING: normal; COLOR: rgb(0,0,0); WORD-SPACING: 0px"
class=Apple-style-span><SPAN
style="FONT-FAMILY: Verdana,Arial,'Bitstream Vera Sans',Helvetica,sans-serif; FONT-SIZE: 13px"
class=Apple-style-span>
<P>screenshot.png does not look like the theme's output.</P></SPAN></SPAN>
<LI><SPAN
style="WIDOWS: 2; TEXT-TRANSFORM: none; TEXT-INDENT: 0px; BORDER-COLLAPSE: separate; FONT: medium 'Times New Roman'; WHITE-SPACE: normal; ORPHANS: 2; LETTER-SPACING: normal; COLOR: rgb(0,0,0); WORD-SPACING: 0px"
class=Apple-style-span><SPAN
style="FONT-FAMILY: Verdana,Arial,'Bitstream Vera Sans',Helvetica,sans-serif; FONT-SIZE: 13px"
class=Apple-style-span>
<P>wp_nav_menu(); being used but registration of available locations not
being set</P></SPAN></SPAN>
<LI><SPAN
style="WIDOWS: 2; TEXT-TRANSFORM: none; TEXT-INDENT: 0px; BORDER-COLLAPSE: separate; FONT: medium 'Times New Roman'; WHITE-SPACE: normal; ORPHANS: 2; LETTER-SPACING: normal; COLOR: rgb(0,0,0); WORD-SPACING: 0px"
class=Apple-style-span><SPAN
style="FONT-FAMILY: Verdana,Arial,'Bitstream Vera Sans',Helvetica,sans-serif; FONT-SIZE: 13px"
class=Apple-style-span>
<P>all scripts and styles should be enqueued ref: actions.php line 288,
line 427-429 suffusion-options-page.php and various other locations,
unless otherwise inapplicable to make the code work properly, i.e. <!--
IF statements</P></SPAN></SPAN></LI></OL></BLOCKQUOTE>Point #1 is incorrect.
You get that with the Magazine template (bundled with the theme, Templates
-> Magazine in the options page) and the Minima skin (Visual Effects ->
Theme Selection, WITHOUT changing any code.<BR><BR>Point #2: The use of
wp_nav_menu() is recommended, not mandatory, as per the Theme Review
checklist:<BR>
<BLOCKQUOTE
style="BORDER-LEFT: rgb(204,204,204) 1px solid; MARGIN: 0pt 0pt 0pt 0.8ex; PADDING-LEFT: 1ex"
class=gmail_quote>
<P>Themes are strongly suggested to support the following customization
features: </P>
<UL>
<LI>Custom Menu - <TT><A title="Function Reference/wp nav menu"
href="http://codex.wordpress.org/Function_Reference/wp_nav_menu">wp_nav_menu()</A></TT>
<UL>
<LI>If a navigation menu is part of the Theme's design, it should be a
customizable menu. For Themes that do not have a navigation menu, users
should be able to add a Custom Menu widget.
</LI></UL></LI></UL></BLOCKQUOTE>If you define a custom WP 3.0 menu you can
include it in the navigation bars from the settings in Blog Features ->
Main Navigation Bar Setup or Blog Features -> Top Navigation Bar Setup. If
you do not define a native WP menu you can still build your menus from the
Navigation bar Setup pages in the options. I don't see how this is grounds for
rejection.<BR><BR>Point #3: The scripts and styles are enqueued where
applicable. The line #288 in actions.php that has been listed is a conditional
include, and the lines 427-429 in suffusion-options-page.php are for scripts
in the admin page, not in the theme itself. The stylesheets cannot be enqueued
because there is a custom-styles.php file that is printed out, and enqueuing
destroys the sequence for that.<BR><BR>IMHO, the review is not thorough
enough.<BR class=Apple-interchange-newline><BR>Sayontan.<BR><BR>
<DIV class=gmail_quote>On Sat, Aug 7, 2010 at 10:26 AM, <SPAN dir=ltr><<A
href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</A>></SPAN>
wrote:<BR>
<BLOCKQUOTE
style="BORDER-LEFT: rgb(204,204,204) 1px solid; MARGIN: 0pt 0pt 0pt 0.8ex; PADDING-LEFT: 1ex"
class=gmail_quote>Thank you for uploading version 3.6.2 of
Suffusion.<BR><BR>Feedback will be provided at <A
href="http://themes.trac.wordpress.org/ticket/604"
target=_blank>http://themes.trac.wordpress.org/ticket/604</A><BR><FONT
color=#888888><BR>--<BR>The WordPress.org Themes Team<BR><A
href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</A><BR></FONT></BLOCKQUOTE></DIV><BR><BR
clear=all><BR>-- <BR>Sayontan Sinha<BR><A
href="http://mynethome.net">http://mynethome.net</A> | <A
href="http://mynethome.net/blog">http://mynethome.net/blog</A><BR>--<BR>Lake
Chargoggagoggmanchauggagoggchaubunagungamaugg - 45-letter Native American name
for Lake Webster, Webster, Massachusetts, meaning "You fish on your side; I
fish on my side; nobody fish in the middle."<BR>
<P>
<HR>
<P></P>_______________________________________________<BR>theme-reviewers
mailing
list<BR>theme-reviewers@lists.wordpress.org<BR>http://lists.wordpress.org/mailman/listinfo/theme-reviewers<BR></BLOCKQUOTE></BODY></HTML>