[theme-reviewers] Question about the review

[Chip Bennett]

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.

Responses are inline.

On Mon, Jun 6, 2011 at 6:16 PM, Daniel Fenn <danielx386 at gmail.com> wrote:

> Hello,
>
> My wordpress theme have been reviewed overnight and I would like to ask
> some questions about the feedback that I been given. link:
> http://themes.trac.wordpress.org/ticket/4095
>
> 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.
>

 http://codex.wordpress.org/Theme_Review#Theme_Template_Files

"screenshot.png

   - Recommended 4:3 W:H ratio, size 300x225px.
   - *Maximum size: 320:240px*"

2) "get_search_form() 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. I know that it can go in the
> searchform.php file, but there is one place where "get_search_form()" can't
> be used.
>

http://codex.wordpress.org/Theme_Review#Theme_Template_Files

"Search Form: *Must be included using
get_search_form()<http://codex.wordpress.org/Function_Reference/get_search_form>
*"


>
> 3) the whole "Theme Settings & Data Security" area - Can someone please
> help me fix up those errors?
>

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.

>
> 4) "Permalinks are not shown for posts without titles." I'm sorry but the
> default wordpress theme does not even meet that requirement.
>

Actually, yes it does. TwentyTen uses the Post timestamp as a Permalink.

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.

Chip
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110606/c3fb1b0c/attachment.htm>