[theme-reviewers] Theme cleanup

[Jose Castaneda]

Any 'clean up' already sounds like plugin territory, especially if it deal
with the <link> tags. Not really presentation.


On Mon, Feb 24, 2014 at 5:25 AM, Balakrishnan <gosubmit at gmail.com> wrote:

> Ola,
>
> Redirecting the search results page is definitely weird :( i wouldn't
> approve the code which has any suspicious behaviour.
> Thanks
>
>
> On Mon, Feb 24, 2014 at 12:15 PM, Daniel <danielx386 at gmail.com> wrote:
>
>> The redirect search is a no no IMO as that really a plug-in thing.
>>
>> On 2/24/14, Ola Łączek <ola at bodera.com> wrote:
>> > Hello!
>> >
>> > I'm reviewing a theme, which has a couple of "clean up" functions, and
>> I'd
>> > like to know if those are allowed:
>> > - Clean up output of stylesheet <link> tags
>> > - Redirects search results from /?s=query to /search/query/, converts
>> %20
>> > to +
>> > - Remove unnecessary self-closing tags on get_avatar(),
>> > post_thumbnail_html()
>> > - Fix for empty search queries redirecting to home page
>> > - Don't return the default description in the RSS feed if it hasn't been
>> > changed
>> >
>> > Best regards,
>> > Ola Laczek
>> >
>>
>>
>> --
>> Regards,
>> Daniel Fenn
>> _______________________________________________
>> 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/20140224/a9d05e8e/attachment.html>