WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#9449 closed defect (bug) (fixed)

Links Widget shows erroneous title + is missing an option

Reported by: Denis-de-Bernardy Owned by:
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.8
Component: Widgets Keywords: has-patch tested commit
Focuses: Cc:

Description

The current Links widget in WP 2.8 trunk displays the name of the first category, rather than the expected All Links.

The attached patch does two things:

  • Adds an extra option that allows to only display links from a specified category
  • Sets the title to All Links in the widgets screen when no category is specified

Attachments (1)

9449.diff (2.8 KB) - added by Denis-de-Bernardy 5 years ago.

Download all attachments as: .zip

Change History (3)

Denis-de-Bernardy5 years ago

comment:1 Denis-de-Bernardy5 years ago

Can anyone commit this?

I'd really like to stop needing to maintain my link widgets plugin, and mark it as "not useful as of WP 2.8".

It's just gory to scan through the widgets one by one, check if it's a text widget, and then replace the various callbacks as needed. (Using the new method, replacing the widget altogether is probably a better option, too...)

The same holds for text widgets (which also has a tried and tested patch): #5113

comment:2 ryan5 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [10896]) Allowing displaying links from a specific category in the links widget. Props Denis-de-Bernardy. fixes #9449

Note: See TracTickets for help on using tickets.