<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
Alina, please post your comments on Trac so the theme author can see
them. If you're not sure what to post, take a look at some of the
other tickets to get an idea of what other reviewers say.<br>
<br>
Make sure you've done a full review of the theme so when/if the
theme author resubmits the theme, (s)he'll be aware of all the
issues you've found, not just the initial ones.<br>
<br>
Chip has put together this guide, and I recommend taking a look if
you're unsure about anything:<br>
<a class="moz-txt-link-freetext" href="http://www.chipbennett.net/2011/04/20/a-guide-to-reviewing-themes-for-the-wordpress-theme-repository/">http://www.chipbennett.net/2011/04/20/a-guide-to-reviewing-themes-for-the-wordpress-theme-repository/</a><br>
<br>
Philip<br>
<br>
<br>
On 4/23/11 4:35 PM, Alinda Pete wrote:
<blockquote cite="mid:865078.27617.qm@web113306.mail.gq1.yahoo.com"
type="cite">
<style type="text/css"><!-- DIV {margin:0px;} --></style>
<div style="font-family: tahoma,'new york',times,serif; font-size:
14pt; color: rgb(255, 0, 127);">
<div><br>
</div>
<div><br>
</div>
<div>Ok, I just ran a check on the theme titled "Monospace"
assigned via tkt#3592 and it came back with the following
errors:</div>
<div><br>
</div>
<div><span class="Apple-style-span" style="color: rgb(51, 51,
51); font-family: 'Lucida Grande',Verdana,Arial,'Bitstream
Vera Sans',sans-serif; font-size: 13px; line-height: 18px;">
<ul class="tc-result" style="margin: 0px 0px 10px 20px;
padding: 0px; border-width: 0px; outline-width: 0px;
background-color: transparent; list-style-type: none;
font-size: 12px;">
<li style="margin: 0px 0px 6px; padding: 0px;
border-width: 0px; outline-width: 0px; background-color:
transparent; line-height: 16px;"><span class="tc-lead
tc-recommended" style="margin: 0px; padding: 0px;
border-width: 0px; outline-width: 0px;
background-color: transparent; color: green;">RECOMMENDED</span>:
could not find the file <strong style="margin: 0px;
padding: 0px; border-width: 0px; outline-width: 0px;
background-color: transparent;">readme.txt</strong> in
the theme. Please see <a moz-do-not-send="true"
href="http://codex.wordpress.org/Theme_Review#Theme_Documentation"
style="margin: 0px; padding: 0px; border-width: 0px;
outline-width: 0px; background-color: transparent;
color: rgb(33, 117, 155);">Theme_Documentation</a> for
more information.</li>
<li style="margin: 0px 0px 6px; padding: 0px;
border-width: 0px; outline-width: 0px; background-color:
transparent; line-height: 16px;"><span class="tc-lead
tc-recommended" style="margin: 0px; padding: 0px;
border-width: 0px; outline-width: 0px;
background-color: transparent; color: green;">RECOMMENDED</span>:
Screenshot is wrong size! Detected: <strong
style="margin: 0px; padding: 0px; border-width: 0px;
outline-width: 0px; background-color: transparent;">300x255px</strong>.
Maximum allowed size is 320x240px.</li>
<li style="margin: 0px 0px 6px; padding: 0px;
border-width: 0px; outline-width: 0px; background-color:
transparent; line-height: 16px;"><span class="tc-lead
tc-recommended" style="margin: 0px; padding: 0px;
border-width: 0px; outline-width: 0px;
background-color: transparent; color: green;">RECOMMENDED</span>:
No reference to nav_menu's was found in the theme. Note
that if your theme has a menu bar, it is required to use
the WordPress nav_menu functionality for it.</li>
<li style="margin: 0px 0px 6px; padding: 0px;
border-width: 0px; outline-width: 0px; background-color:
transparent; line-height: 16px;"><span class="tc-lead
tc-recommended" style="margin: 0px; padding: 0px;
border-width: 0px; outline-width: 0px;
background-color: transparent; color: green;">RECOMMENDED</span>:
No reference to <strong style="margin: 0px; padding:
0px; border-width: 0px; outline-width: 0px;
background-color: transparent;">add_editor_style()</strong> 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.</li>
<li style="margin: 0px 0px 6px; padding: 0px;
border-width: 0px; outline-width: 0px; background-color:
transparent; line-height: 16px;"><span class="tc-lead
tc-recommended" style="margin: 0px; padding: 0px;
border-width: 0px; outline-width: 0px;
background-color: transparent; color: green;">RECOMMENDED</span>:
No reference to <strong style="margin: 0px; padding:
0px; border-width: 0px; outline-width: 0px;
background-color: transparent;">add_custom_image_header</strong> was
found in the theme. It is recommended that the theme
implement this functionality if using an image for the
header.</li>
<li style="margin: 0px 0px 6px; padding: 0px;
border-width: 0px; outline-width: 0px; background-color:
transparent; line-height: 16px;"><span class="tc-lead
tc-recommended" style="margin: 0px; padding: 0px;
border-width: 0px; outline-width: 0px;
background-color: transparent; color: green;">RECOMMENDED</span>:
No reference to <strong style="margin: 0px; padding:
0px; border-width: 0px; outline-width: 0px;
background-color: transparent;">add_custom_background()</strong> was
found in the theme. If the theme uses background images
or solid colors for the background, then it is
recommended that the theme implement this functionality.</li>
<li style="margin: 0px 0px 6px; padding: 0px;
border-width: 0px; outline-width: 0px; background-color:
transparent; line-height: 16px;"><span class="tc-lead
tc-info" style="margin: 0px; padding: 0px;
border-width: 0px; outline-width: 0px;
background-color: transparent; color: blue;">INFO</span>:
Possible hard-coded links were found in the file <strong
style="margin: 0px; padding: 0px; border-width: 0px;
outline-width: 0px; background-color: transparent;">footer.php</strong>.
<pre class="tc-grep" style="margin: 0px; padding: 0px; border-width: 0px; outline-width: 0px; background-color: transparent; white-space: pre-wrap;">Line 20: <span><a href='<span class="tc-grep" style="margin: 0px; padding: 0px; border-width: 0px; outline-width: 0px; background-color: yellow;"><span><a moz-do-not-send="true" target="_blank" href="http://vinicius.soylocoporti.org.br/monospace-wordpress-theme">http://vinicius.soylocoporti.org.br/monospace-wordpress-theme</a></span></span>'>Monospace Wo</pre>
<pre class="tc-grep" style="margin: 0px; padding: 0px; border-width: 0px; outline-width: 0px; background-color: transparent; white-space: pre-wrap;">
</pre>
<pre class="tc-grep" style="margin: 0px; padding: 0px; border-width: 0px; outline-width: 0px; background-color: transparent; white-space: pre-wrap;">Which I erred on the side of caution to paste the outputted messages.</pre>
<pre class="tc-grep" style="margin: 0px; padding: 0px; border-width: 0px; outline-width: 0px; background-color: transparent; white-space: pre-wrap;">
</pre>
<pre class="tc-grep" style="margin: 0px; padding: 0px; border-width: 0px; outline-width: 0px; background-color: transparent; white-space: pre-wrap;">If you are breezing through, theres no ref to nav menu, there is no readme.txt found in the theme(I double checked)</pre>
<pre class="tc-grep" style="margin: 0px; padding: 0px; border-width: 0px; outline-width: 0px; background-color: transparent; white-space: pre-wrap;">Screenshot</pre>
<pre class="tc-grep" style="margin: 0px; padding: 0px; border-width: 0px; outline-width: 0px; background-color: transparent; white-space: pre-wrap;">
</pre>
<pre class="tc-grep" style="margin: 0px; padding: 0px; border-width: 0px; outline-width: 0px; background-color: transparent; white-space: pre-wrap;">I would not approve this theme just yet - there are a few issues that should be resolved prior to release. </pre>
</li>
</ul>
</span></div>
</div>
<pre wrap="">
<fieldset class="mimeAttachmentHeader"></fieldset>
_______________________________________________
theme-reviewers mailing list
<a class="moz-txt-link-abbreviated" href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a>
<a class="moz-txt-link-freetext" href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>
</pre>
</blockquote>
</body>
</html>