[theme-reviewers] Adding warnings to Theme-Check for remove_filter on wpautop and wptexturize

[Daniel Tara]

Wow! So this is from where all those themes on Themeforest took their [three_columns_last] shortcodes. I was always wondering how can this method be so inefficient and yet everybody seems to prefer it. Now it's clear: they were following a tutorial. I can give you dozens of examples of themes that do it just this way, every once in a while I stumble upon one. I can understand now the level of frustration plugin developers must have.

Daniel

On Sep 6, 2012, at 4:02 AM, Justin Tadlock wrote:

> That tutorial looked like it was written by someone who doesn't understand what the `$atts` parameter does in his own code.  22 shortcodes for the work of 1.  I bet that's a huge selling point for some themes. 
> 
> I need to write a plugin to do that, just better (shouldn't be in a theme anyway).  I really think I might do this.
> 
> I could tell this tutorial was going nowhere before getting to step #3.
> 
> On 9/5/2012 7:50 PM, Japheth Thomson wrote:
>> Yeah, it seems strange. There's a tutorial here: http://tutorials.mysitemyway.com/adding-column-layout-shortcodes-to-a-wordpress-theme/#highlighter_671922 which seems to claim that the <p></p> and <br /> tags mess with column shortcodes, or something like that?
>> 
>> 
>> On 6 September 2012 10:47, Justin Tadlock <justin at justintadlock.com> wrote:
>> I can't really think of a reason a theme should be doing either of those things at all.  There might be some legitimate uses (from a theme perspective) but none come to mind at the moment.
>> 
>> 
>> On 9/5/2012 7:27 PM, Japheth Thomson wrote:
>>> Hello,
>>> 
>>> There's been some discussion today on Twitter, and a blog post I wrote ( http://japh.co/NSXyTr ), about themes that run code like:
>>> 
>>> remove_filter( 'the_content', 'wpautop' );
>>> remove_filter( 'the_content', 'wptexturize' );
>>> 
>>> And also implementing a [raw][/raw] shortcode like http://wordpress.org/support/topic/plugin-remove-wpautop-wptexturize-with-a-shortcode
>>> 
>>> Could we add checks for these things into Theme-Check? And would it be sensible to do so?
>>> 
>>> Cheers,
>>> Japh
>>> 
>>> -- 
>>> Japh Thomson
>>> WordPress Evangelist
>>> japh at envato.com
>>> http://envato.com
>>> http://twitter.com/envato
>>> 
>>> 
>>> 
>>> 
>>> _______________________________________________
>>> 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
>> 
>> 
>> 
>> 
>> -- 
>> Japh Thomson
>> WordPress Evangelist
>> japh at envato.com
>> http://envato.com
>> http://twitter.com/envato
>> 
>> 
>> 
>> 
>> _______________________________________________
>> 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/20120906/77bcdfe2/attachment.htm>