Make WordPress Core

Opened 18 years ago

Closed 17 years ago

#4259 closed enhancement (fixed)

Text & page widget improvemets

Reported by: denis-de-bernardy's profile Denis-de-Bernardy Owned by: rob1n's profile rob1n
Milestone: 2.2.2 Priority: normal
Severity: normal Version: 2.2
Component: Administration Keywords: has-patch
Focuses: Cc:

Description (last modified by rob1n)

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

Attachments (4)

widgets.png (67.7 KB) - added by Denis-de-Bernardy 18 years ago.
widgets.diff (3.5 KB) - added by Denis-de-Bernardy 18 years ago.
4259.diff (3.8 KB) - added by rob1n 18 years ago.
page order + text widget filters
page-exclude.diff (2.5 KB) - added by rob1n 18 years ago.
option to exclude pages

Download all attachments as: .zip

Change History (31)

#1 @matt
18 years ago

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

#2 @rob1n
18 years ago

  • Owner changed from anonymous to rob1n
  • Status changed from new to assigned
  • Summary changed from Has anyone bothered to test widgets in Internet Explorer? to Text & page widget improvemets

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

#3 @rob1n
18 years ago

Open another ticket for the IE 7 bug.

#4 @rob1n
18 years ago

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

#5 @rob1n
18 years ago

  • Keywords has-patch added

#6 @rob1n
18 years ago

  • Priority changed from high to normal
  • Severity changed from blocker to normal
  • Type changed from defect to enhancement

#7 @ryan
18 years ago

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.

#8 @rob1n
18 years ago

  • Milestone changed from 2.2 to 2.2.1

#9 @rob1n
18 years ago

(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
18 years ago

page order + text widget filters

#10 follow-up: @Denis-de-Bernardy
18 years ago

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.

#11 @Denis-de-Bernardy
18 years ago

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.

#12 in reply to: ↑ 10 @rob1n
18 years ago

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 :).

#13 @ryan
18 years ago

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.

#14 @rob1n
18 years ago

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

#15 @ryan
18 years ago

Ideally, it should just work.

#16 @rob1n
18 years ago

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

#17 @ryan
18 years ago

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

#18 @ryan
18 years ago

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

#19 @ryan
18 years ago

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.

#20 @rob1n
18 years ago

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

@rob1n
18 years ago

option to exclude pages

#21 @rob1n
18 years ago

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

#22 @rob1n
18 years ago

  • Description modified (diff)

#23 @rob1n
18 years ago

  • Milestone changed from 2.2.1 to 2.2.2

#24 follow-up: @rob1n
18 years ago

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

#25 in reply to: ↑ description @drmike
18 years ago

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. :)

#26 in reply to: ↑ 24 @foolswisdom
17 years ago

  • Milestone changed from 2.2.2 to 2.2.3

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?

#27 @foolswisdom
17 years ago

  • Milestone changed from 2.2.3 to 2.2.2
  • Resolution set to fixed
  • Status changed from assigned to closed

Closing as fixed.

Note: See TracTickets for help on using tickets.