WordPress.org
Get WordPress

Make WordPress Core

Opened 6 weeks ago

Closed 5 weeks ago

#64330 closed enhancement (fixed)

Localized item separator on admin screens

Reported by: geminorum's profile geminorum Owned by: westonruter's profile westonruter
Milestone: 7.0 Priority: normal
Severity: normal Version: 6.9
Component: I18N Keywords: good-first-bug has-patch
Focuses: rtl, administration Cc:

Description

Posts/Media/Sites states renderer failed to use the localized separator provided via wp_get_list_item_separator()

Attachments (2)

incorrect.png​ (11.3 KB) - added by geminorum 6 weeks ago.
incorrect use of separator
correct.png​ (11.4 KB) - added by geminorum 6 weeks ago.
correct use of separator

Download all attachments as: .zip

Change History (7)

@geminorum
6 weeks ago

incorrect use of separator

@geminorum
6 weeks ago

correct use of separator

#1 @SergeyBiryukov
6 weeks ago

  • Keywords needs-patch good-first-bug added
  • Milestone changed from Awaiting Review to 7.0

This ticket was mentioned in ​PR #10577 on ​WordPress/wordpress-develop by ​@geminorum.
6 weeks ago #2

  • Keywords has-patch added; needs-patch removed

Trac ticket: https://core.trac.wordpress.org/ticket/64330

#3 @westonruter
6 weeks ago

  • Owner set to westonruter
  • Status changed from new to reviewing

​@westonruter commented on ​PR #10577:
5 weeks ago #4

Before:

https://github.com/user-attachments/assets/318431aa-4904-478b-ac83-5b252bb98ee7

After:

https://github.com/user-attachments/assets/23938e04-d0d8-41e1-95d7-4b622bdf395e

#5 @westonruter
5 weeks ago

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

In 61371:

I18N: Use wp_get_list_item_separator() as list separator for the states of sites, posts, and media.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/10577

Follow-up to [52929].

Props geminorum.
See #39733.
Fixes #64330.

Note: See TracTickets for help on using tickets.