For future reference, these questions should *first* be asked *in the ticket*, so that the Reviewer can help answer them, and help you resolve any issues.<div><br></div><div>Responses are inline.<br><br><div class="gmail_quote">
On Mon, Jun 6, 2011 at 6:16 PM, Daniel Fenn <span dir="ltr"><<a href="mailto:danielx386@gmail.com">danielx386@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div>Hello,</div>
<div> </div>
<div>My wordpress theme have been reviewed overnight and I would like to ask some questions about the feedback that I been given. link: <a href="http://themes.trac.wordpress.org/ticket/4095" target="_blank">http://themes.trac.wordpress.org/ticket/4095</a></div>
<div> </div>
<div>1) "Screenshot is wrong size. Should be a max of 320 X 240" Why am I being knocked back by that? It said "should", not "required". If it was required then I would understand.</div></blockquote>
<div><br></div><div> <a href="http://codex.wordpress.org/Theme_Review#Theme_Template_Files">http://codex.wordpress.org/Theme_Review#Theme_Template_Files</a></div><div><br></div><div>"<span class="Apple-style-span" style="font-family: 'Lucida Grande', Verdana, 'Bitstream Vera Sans', Arial, sans-serif; font-size: 12px; line-height: 22px; background-color: rgb(255, 255, 255); ">screenshot.png</span></div>
<span class="Apple-style-span" style="font-family: 'Lucida Grande', Verdana, 'Bitstream Vera Sans', Arial, sans-serif; font-size: 12px; line-height: 22px; background-color: rgb(255, 255, 255); "><ul style="font-variant: normal !important; text-align: left !important; margin-top: 0px; margin-right: 0px; margin-bottom: 22px; margin-left: 16px; padding-top: 0px; padding-right: 0px; padding-bottom: 0px; padding-left: 0px; list-style-type: square; list-style-position: initial; list-style-image: initial; ">
<li style="font-variant: normal !important; text-align: left !important; margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; padding-top: 0px; padding-right: 0px; padding-bottom: 0px; padding-left: 0px; ">
Recommended 4:3 W:H ratio, size 300x225px.</li><li style="font-variant: normal !important; text-align: left !important; margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; padding-top: 0px; padding-right: 0px; padding-bottom: 0px; padding-left: 0px; ">
<b>Maximum size: 320:240px</b><span class="Apple-style-span" style="font-family: arial; line-height: normal; font-size: small; ">"</span><span class="Apple-style-span" style="font-family: arial; line-height: normal; font-size: small; "> </span></li>
</ul></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div>2) "<font face="Courier New">get_search_form()</font> should be used to include search form. " Again, why am I being knocked back by that? It said "should", not "required". If it was required then I would understand. <font face="Courier New">I know that it can go in the searchform.php file, but there is one place where "get_search_form()" can't be used.</font></div>
</blockquote><div><br></div><div><a href="http://codex.wordpress.org/Theme_Review#Theme_Template_Files">http://codex.wordpress.org/Theme_Review#Theme_Template_Files</a></div><div><br></div><div>"<span class="Apple-style-span" style="font-family: 'Lucida Grande', Verdana, 'Bitstream Vera Sans', Arial, sans-serif; font-size: 12px; line-height: 22px; background-color: rgb(255, 255, 255); ">Search Form: <b>Must be included using <tt style="font-variant: normal !important; text-align: left !important; "><a href="http://codex.wordpress.org/Function_Reference/get_search_form" title="Function Reference/get search form" style="font-variant: normal !important; text-align: left !important; text-decoration: none; color: rgb(33, 117, 155); ">get_search_form()</a></tt></b></span>"</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><font face="Courier New"></font> </div>
<div>3) the whole "Theme Settings & Data Security" area - Can someone please help me fix up those errors?</div></blockquote><div><br></div><div>I would suggest asking specific questions in the ticket, regarding the specific issues you're having difficulties addressing/implementing. I've CCed myself on the ticket, so I'll see any questions you ask there. </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div> </div>
<div>4) "Permalinks are not shown for posts without titles." I'm sorry but the default wordpress theme does not even meet that requirement.</div></blockquote><div><br></div><div>Actually, yes it does. TwentyTen uses the Post timestamp as a Permalink.</div>
<div><br></div><div>Besides, "but Theme X does/doesnt..." is not a valid argument. Even if TwentyTen (or soon, TwentyEleven) didn't meet some Guideline, the Theme Review Team has no direct control over that.</div>
<div><br></div><div>Chip</div></div></div>