Make WordPress Core

Opened 7 years ago

Last modified 3 years ago

#39106 assigned enhancement

Make hierarchy level indicator text changeable [WP List]

Reported by: gkloveweb's profile gk.loveweb Owned by:
Milestone: Future Release Priority: low
Severity: minor Version: 4.6.1
Component: Posts, Post Types Keywords: good-first-bug has-patch
Focuses: ui, administration Cc:


In WP_Posts_List_Table class, dashes(—) used to indicate the hierarchy level of post item, which is not changeble by any filter, we can add a new filter to replace this string.

$pad = str_repeat( '— ', $this->current_level );


Attachments (2)

39106.patch (1.1 KB) - added by rebasaurus 4 years ago.
Adding filter 'post_title_child_separator'
39106.diff (998 bytes) - added by mayankmajeji 4 years ago.

Download all attachments as: .zip

Change History (9)

#1 @gk.loveweb
7 years ago

  • Type changed from feature request to defect (bug)

#2 @valentinbora
4 years ago

  • Keywords good-first-bug needs-patch added
  • Milestone changed from Awaiting Review to Future Release
  • Owner set to valentinbora
  • Priority changed from normal to low
  • Severity changed from normal to minor
  • Status changed from new to accepted
  • Type changed from defect (bug) to enhancement

#3 @valentinbora
4 years ago

  • Component changed from Administration to Posts, Post Types
  • Owner valentinbora deleted
  • Status changed from accepted to assigned

Thanks for the ticket @gkloveweb. I believe it makes sense to allow this to be tweakable so I've marked it accordingly.

It also seems like an easy to do change, so I've marked it as good-first-bug for new contributors to tackle.

4 years ago

Adding filter 'post_title_child_separator'

#4 @rebasaurus
4 years ago

  • Keywords has-patch added; needs-patch removed

#5 @seanchayes
4 years ago

I downloaded and tested this patch.

It worked as expected and I was easily able to change the separator used for the hierarchy post separator by adding the filter to my functions.php and trying out different combinations.

One code change I would suggest is to rename name the variable used from $pad to $separator - as I believe this would more closely match the name of the suggested filter naming.

4 years ago


#6 @mayankmajeji
4 years ago

I completely agree with @seanchayes and changed $pad to $separator as it makes more sense and semantically correct. I also cross-checked for the conflicts and find zero conflicts.

#7 @10010110
3 years ago

I would suggest an even simpler fix without the need of a filter: just add an element around the character so that it can be accessed by a CSS selector, and have the user add custom admin CSS to hide it (or whatever), like so:

$pad = str_repeat( '<span class="dash">&#8212; </span>', $this->current_level );
Note: See TracTickets for help on using tickets.