Just throwing my hat into the ring on the OP; and, simply put: no, not acceptable ... way too many ways for it to be exploited. Does it need to be written into the guidelines, not really ... as noted, not everything possible needs to be written into the guidelines, some things are just common sense approaches.<br>
<br>Now, to the code itself: very nice ... perhaps used in the Theme documentation to give the end-user a better perspective of what can be done with the "right" content.<br><br>Otherwise, any items not currently showing well in the current preview content should be addressed. Personally I believe the preview content should be the same as the Theme Unit Test data ... not much reason why it shouldn't in my mind.<br>
<br><br>Cais.<br><br><div class="gmail_quote">On Wed, May 11, 2011 at 9:55 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
That will require having a Theme available on <a href="http://wordpress.com" target="_blank">wordpress.com</a> that adds support for all Post Format types. Remember: the "master" Theme Unit Test data are here: <a href="http://wpthemetestdata.wordpress.com" target="_blank">http://wpthemetestdata.wordpress.com</a><div>
<br></div><div>(Yes, this is a limitation. And yes: it is outside of our control.)</div><div><br></div><div><font color="#888888">Chip</font><div><div></div><div class="h5"><br><br><div class="gmail_quote">On Wed, May 11, 2011 at 8:40 PM, James Laws <span dir="ltr"><<a href="mailto:jamielaws@gmail.com" target="_blank">jamielaws@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think one adjustment might be to actually add the appropriate post_formats to the posts that claim to be for them. Otherwise they just look like ordinary posts. I had to do this manually on my data so that post_formats would display properly.<div>
<br></div><div>Unless my import doesn't import correctly, which is the case for some of the html with the comments test. :(</div><div><br></div><div><font color="#888888">James<br><br></font><div class="gmail_quote">
<div>On Wed, May 11, 2011 at 9:35 PM, Ian Stewart <span dir="ltr"><<a href="mailto:ian@iandanielstewart.com" target="_blank">ian@iandanielstewart.com</a>></span> wrote:<br>
</div><div><div></div><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On 2011-05-11, at 5:46 PM, Chip Bennett wrote:<br>
<br>
> If the Theme previewer needs to be fixed/improved, then let's do that.<br>
<br>
</div>This might be a good time to look at using the same content we use to test themes as the preview content in /extend/themes (which might mean adjusting the test content).<br>
<font color="#888888"><br>
Ian<br>
</font><div><div></div><div><br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
</div></div></blockquote></div></div></div><br></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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></div></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>