Opened 8 years ago
Last modified 4 years ago
#39106 assigned enhancement
Make hierarchy level indicator text changeable [WP List]
Reported by: | 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: |
Description
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 );
Source: http://wordpress.stackexchange.com/questions/248405/replace-dashes-before-title-in-page-list
Attachments (2)
Change History (9)
#2
@
5 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
@
5 years ago
- Component changed from Administration to Posts, Post Types
- Owner valentinbora deleted
- Status changed from accepted to assigned
#5
@
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.
#6
@
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
@
4 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">— </span>', $this->current_level );
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.