[theme-reviewers] Cirumventing Theme Directory Preview in Ticket #3787

[Edward Caissie]

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.

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.

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.


Cais.

On Wed, May 11, 2011 at 9:55 PM, Chip Bennett <chip at chipbennett.net> wrote:

> That will require having a Theme available on wordpress.com that adds
> support for all Post Format types. Remember: the "master" Theme Unit Test
> data are here: http://wpthemetestdata.wordpress.com
>
> (Yes, this is a limitation. And yes: it is outside of our control.)
>
> Chip
>
>
> On Wed, May 11, 2011 at 8:40 PM, James Laws <jamielaws at gmail.com> wrote:
>
>> 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.
>>
>> Unless my import doesn't import correctly, which is the case for some of
>> the html with the comments test. :(
>>
>> James
>>
>> On Wed, May 11, 2011 at 9:35 PM, Ian Stewart <ian at iandanielstewart.com>wrote:
>>
>>> On 2011-05-11, at 5:46 PM, Chip Bennett wrote:
>>>
>>> > If the Theme previewer needs to be fixed/improved, then let's do that.
>>>
>>> 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).
>>>
>>> Ian
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110511/696d87fc/attachment-0001.htm>