[theme-reviewers] Theme cleanup

Balakrishnan gosubmit at gmail.com
Mon Feb 24 13:25:00 UTC 2014


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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20140224/b15e1f2d/attachment.html>


More information about the theme-reviewers mailing list