3804 posts
  • Elite Author
  • Author Level 11
  • Trendsetter
  • 7 Years of Membership
+12 more
KrownThemes
says

I am aware of all the answers from the other threads regarding this issue(especially bitfade’s class and the tutorial with the shortcodes filter order). I’m just trying an alternative and wanted to be sure that i can pass review:

 function rbFilterShortcodes($content){
     $filtered = str_replace(
         array(
             'autop"></p>',
             '<p></p>',
             '<p></div>'
         ),
         array(
             'autop">',
             '',
             '</div>'
         ),
         $content);
     return $filtered;
 }

add_filter("the_content", "rbFilterShortcodes", 99);

All of my shortcodes have the “autop” class at the end, so this is how i’m searching for them.. Also, there are more weird stuff happening, such as the last two filters..

Is it safe to work like this?

3804 posts
  • Elite Author
  • Author Level 11
  • Trendsetter
  • 7 Years of Membership
+12 more
KrownThemes
says

3804 posts
  • Elite Author
  • Author Level 11
  • Trendsetter
  • 7 Years of Membership
+12 more
KrownThemes
says

bump! :D

198 posts Dreams Money Can Buy
  • Top Monthly Author
  • Trendsetter
  • Weekly Top Seller
  • Author Level 12
+8 more
SwiftIdeas
says

I’m not sure if the reviewers would have anything against it, but does it work well for you?

- Ed

3804 posts
  • Elite Author
  • Author Level 11
  • Trendsetter
  • 7 Years of Membership
+12 more
KrownThemes
says

I’m not sure if the reviewers would have anything against it, but does it work well for you? - Ed

Thanks :)

It works really well for me.. Actually, it works the best, since there are always some weird paragraphs appearing before or after elements.. So i can also fix stuff like: <p></section> for example..

198 posts Dreams Money Can Buy
  • Top Monthly Author
  • Trendsetter
  • Weekly Top Seller
  • Author Level 12
+8 more
SwiftIdeas
says

I have had issues before where shortcodes on the same line as a paragraph would cause erroneous tags. Seems that it would also be better as it wouldn’t interfere with any 3rd party shortcodes!

So for every shortcode you create, you have a class on the outer wrap of “autop”?

Might need to take a little look at re-working the way we do it ;)

- Ed

3804 posts
  • Elite Author
  • Author Level 11
  • Trendsetter
  • 7 Years of Membership
+12 more
KrownThemes
says

So for every shortcode you create, you have a class on the outer wrap of “autop”?

yep .. :D

38 posts
  • Affiliate Level 3
  • Author Level 7
  • Beta Tester
  • Collector Level 2
+4 more
wpstrong
says

My way is using jQuery :

    
//Clean empty p elements

$('#content p:empty').remove();

It helps a bit

by
by
by
by
by
by