WordPress.org

Make WordPress Core

Opened 7 days ago

Last modified 4 days ago

#48286 new defect (bug)

Avoid use of mdash in starter content for day separator Monday—Friday

Reported by: garrett-eclipse Owned by:
Milestone: 5.4 Priority: normal
Severity: normal Version:
Component: I18N Keywords: good-first-bug has-patch
Focuses: Cc:
PR Number:

Description

In the starter content there's a mix of mdash and ndash when it should only be ndash as the separator in dates and times.
https://github.com/WordPress/wordpress-develop/blob/7a682e690e4915411777080ebc2999da69ad3a39/src/wp-includes/theme.php#L2058

The ndash standardly is the "through" indicator for showing inclusive dates and numbers.
While the mdash is an interruptor and used as an aside.

As such the content - Monday—Friday: 9:00AM–5:00PM
Should simply be - Monday–Friday: 9:00AM–5:00PM

Originally opened here;
https://github.com/WordPress/twentytwenty/issues/830

Attachments (1)

48286.diff (892 bytes) - added by mikejdent 4 days ago.

Download all attachments as: .zip

Change History (3)

#1 @SergeyBiryukov
6 days ago

  • Milestone changed from Awaiting Review to 5.4
  • Summary changed from Avoid use of mdash in starter content for day seperator Monday—Friday to Avoid use of mdash in starter content for day separator Monday—Friday

@mikejdent
4 days ago

#2 @mikejdent
4 days ago

  • Keywords has-patch added

Updated to use ndash instead of mdash

Note: See TracTickets for help on using tickets.