Make WordPress Core

Opened 5 months ago

Last modified 5 months ago

#62138 new defect (bug)

The theme count in theme-install should use a thousand separator

Reported by: dsas's profile dsas Owned by:
Milestone: Awaiting Review Priority: normal
Severity: minor Version: 6.7
Component: Themes Keywords: has-patch
Focuses: Cc:

Description

The count of themes is currently set as a hardcoded number like "6915", when it should (locale dependant) display using a thousands separator like "6,915" to make it easier to read.

Attachments (1)

Screenshot 2024-09-30 at 16.46.41.png (489.2 KB) - added by dsas 5 months ago.
Screenshot for clarity

Download all attachments as: .zip

Change History (2)

@dsas
5 months ago

Screenshot for clarity

This ticket was mentioned in PR #7462 on WordPress/wordpress-develop by @dsas.


5 months ago
#1

  • Keywords has-patch added

The theme navigation includes a count of themes matching the current query. It's currently displayed as a straight number e.g. "6916". It is slightly nicer if the number is displayed with a locale-specific thousands separator.

Before | After


https://github.com/user-attachments/assets/4873438c-45ce-4cb8-9ac2-00f69f1f35ee | https://github.com/user-attachments/assets/53b3a246-dcfa-4fcd-b259-476134d8ae3e

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

Note: See TracTickets for help on using tickets.