Good evening,<div><br></div><div>First, I must point out: TwentyTen is *not* the "perfect sample" Theme. It is merely the current default WordPress Theme. It was released before the current Theme Review Guidelines went into effect. That said, I'll try to answer your questions:</div>
<div><br></div><div>1) I would question whether the Theme output for a non-default [gallery] shortcode should be cause for not approving the Theme. If the Theme outputs properly with the default shortcode (i.e. 3 columns), I would argue that it should be acceptable.</div>
<div><br></div><div>2) Probably, wp_link_pages() is called inside the same container as the_content(), and is not wrapped in a container that clears floats, causing the post-page links to float into the content. Try adding "clear:both;" to div.page-link.</div>
<div><br></div><div>3) It means that custom CSS should be added using the appropriate hook, rather than hard-coded into the document head. For example, add a add_action( 'wp_enqueue_scripts', 'my-custom-css' ) where 'my-custom-css()' defines your custom scripts.</div>
<div><br></div><div>4) Again, TwentyTen is *not* a "perfect sample" Theme. There is no guarantee that it will meet any given guideline (though we certainly hope it does!).</div><div><br></div><div>Chip<br><br><div class="gmail_quote">
On Mon, Dec 20, 2010 at 5:09 PM, Ford Mon <span dir="ltr"><<a href="mailto:fordmon@gmail.com">fordmon@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse"><a href="http://themes.trac.wordpress.org/ticket/2122" style="color:rgb(0, 0, 204)" target="_blank">http://themes.trac.wordpress.org/ticket/2122</a> </span><div>
<br></div><div><br></div><div>Hello All,</div><div><br></div><div><span style="font-family:'Lucida Grande', Verdana, 'Bitstream Vera Sans', Arial, sans-serif;font-size:12px;color:rgb(51, 51, 51);line-height:22px"><p style="margin-top:0px;margin-right:0px;margin-bottom:11px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px">
I have created a theme<br><a href="http://www.themesfromrobin.com/2010/11/black-queen-wp-theme/" rel="nofollow" style="text-decoration:none;color:rgb(33, 117, 155);font-weight:normal;border-bottom-width:1px;border-bottom-style:solid;border-bottom-color:rgb(223, 223, 223)" target="_blank">http://www.themesfromrobin.com/2010/11/black-queen-wp-theme/</a><br>
and uploaded into <a href="http://trac.wordpress.org" target="_blank">trac.wordpress.org</a>. It was tested recently and rejected until I fix the following:</p><p style="margin-top:0px;margin-right:0px;margin-bottom:11px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px">
1) // Galleries with more than three columns aren't shown correctly. Test [gallery columns="4"] and it breaks the gallery.» //</p><p style="margin-top:0px;margin-right:0px;margin-bottom:11px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px">
I looked into the stadard wordpress theme and when tested it there was the same error.<br><a href="http://img507.imageshack.us/img507/6216/screenshot117.png" rel="nofollow" style="text-decoration:none;color:rgb(33, 117, 155);font-weight:normal;border-bottom-width:1px;border-bottom-style:solid;border-bottom-color:rgb(223, 223, 223)" target="_blank">http://img507.imageshack.us/img507/6216/screenshot117.png</a><br>
It can't look well with 4 colums, the best look is 3 columns. Could you explain what to do in this case?</p><p style="margin-top:0px;margin-right:0px;margin-bottom:11px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px">
2. // Floated elements aren't cleared in the post content. This only seems to be the case when usingwp_link_pages().»//</p><p style="margin-top:0px;margin-right:0px;margin-bottom:11px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px">
I can not decipher this statement. Could you help</p><p style="margin-top:0px;margin-right:0px;margin-bottom:11px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px">3. //«Additional stylesheets should be appropriately loaded using wp_enqueue_style()»//</p>
<p style="margin-top:0px;margin-right:0px;margin-bottom:11px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px">What does it mean?</p><p style="margin-top:0px;margin-right:0px;margin-bottom:11px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px">
The main style in my theme is the same as it is in the standard wp theme ( the perfect sample). An additional style is also added according to the codex. At least it is the same as it's in this accepted theme<a href="http://wordpress.org/extend/themes/fudo" rel="nofollow" style="text-decoration:none;color:rgb(33, 117, 155);font-weight:normal;border-bottom-width:1px;border-bottom-style:solid;border-bottom-color:rgb(223, 223, 223)" target="_blank">http://wordpress.org/extend/themes/fudo</a></p>
<p style="margin-top:0px;margin-right:0px;margin-bottom:11px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px">Many thanks in advance</p></span></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>