Text & page widget improvemets

[Denis-de-Bernardy]

XP SP2, IE7

The errors that the attached screenshot will highlight have been around on wordpress.com for longer than I can remember. I'm surprised noone has ever bothered to file them.

In addition, please be so kind to apply the attached patch. It does the following:

1. add an option to wpautop text widgets
2. add an option to exclude pages from the pages widget
3. sort page widgets using menu_order rather than by their title

[matt]

If your patch doesn't fix the IE7 issue, you should create a new ticket for it, and explain the reasoning behind it.

[rob1n]

So your patch doesn't fix errors in IE 7? ...

[rob1n]

Open another ticket for the IE 7 bug.

[rob1n]

Also, we should wp_texturize() the text widget's contents.

[ryan]

wp_list_pages() should sort by menu_order,post_title by default. As for the options, I'd rather wait until 2.2.x. We're planning to add a bunch of options to the widgets.

[rob1n]

(In [5486]) Texturize and autop the text widgets, and allow pages to be sorted by a specified field. Props Denis de Bernardy. see #4259

[rob1n]

page order + text widget filters

[Denis-de-Bernardy]

if you apply the filter 'the_content', you'll get plenty bugs with plugins along the lines of podpress. e.g. if the last post in the loop has media attached to it, each text widget will have a few media links.

also, the page exclude is heavily requested (at least among the users I'm babysitting).

D.

[Denis-de-Bernardy]

lastly, in the text widget's title, if the title is empty, it is turned into a non-breaking space. this defeats the purpose of:

<?php !empty( $title ) ? print($before_title . $title . $after_title) : null; ?>

D.

[rob1n]

Replying to Denis-de-Bernardy:

if you apply the filter 'the_content', you'll get plenty bugs with plugins along the lines of podpress. e.g. if the last post in the loop has media attached to it, each text widget will have a few media links.

Makes sense. I guess we can make the text widget its own filter.

also, the page exclude is heavily requested (at least among the users I'm babysitting).

Next thing I'm looking at :).

[ryan]

autop and texturize are breaking the Google Reader Clip snippet. script tags are peed on and the script src is texturized. Since text widgets are commonly used for such snippets, we need to make sure we don't break them.

[rob1n]

Which brings me back to my original proposal for making texturize/autop a checkbox :).

[ryan]

Ideally, it should just work.

[rob1n]

(In [5509]) Don't show title if user leaves it blank. see #4259

[ryan]

(In [5517]) Remove the_content filter from widget text. see #4259

[ryan]

(In [5518]) Remove the_content filter from widget text. see #4259

[ryan]

Some plugins append things to the_content making it unsuitable for widget use. We can add a widget_text or generic text_content filter. I'd like to get autop and texturize playing nicely first.

[rob1n]

(In [5519]) Run it through the filter. see #4259

[rob1n]

option to exclude pages

[rob1n]

(In [5605]) Add option to exclude pages to pages widget. see #4259

[rob1n]

Okay, all that's left is autop and texturize.

[drmike]

Replying to Denis-de-Bernardy:

The errors that the attached screenshot will highlight have been around on wordpress.com for longer than I can remember. I'm surprised noone has ever bothered to file them.

*chuckle* We have. :)

[foolswisdom]

Replying to rob1n:

Okay, all that's left is autop and texturize.

Maybe, open a new ticket and close this one b/c it is in a released version?

[foolswisdom]

Closing as fixed.