WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#32047 closed enhancement (fixed)

Merge two strings from theme details dialog

Reported by: SergeyBiryukov Owned by: SergeyBiryukov
Priority: low Milestone: 4.3
Component: Themes Version: 4.2
Severity: trivial Keywords: good-first-bug has-patch
Cc: Focuses: accessibility

Description

We have two strings for the Close button in theme details dialog:

Can we use the latter for both?

Attachments (1)

32047.patch​ (834 bytes ) - added by McGuive7 11 years ago.
Switch themes screenreader text to match customizer: "Close details dialog"

Download all attachments as: .zip

Change History (5)

#1 @SergeyBiryukov
11 years ago

  • Priority normal → low
  • Severity normal → trivial

#2 @obenland
11 years ago

  • Focuses accessibility added
  • Keywords needs-patch good-first-bug added
  • Milestone Awaiting Review → 4.3

I think that would make sense.

Adding a11y focus since it's screen reader text.

@McGuive7
11 years ago

Switch themes screenreader text to match customizer: "Close details dialog"

#3 @McGuive7
11 years ago

  • Keywords has-patch added; needs-patch removed

And. . . added. This patch simply changes the screenreader text in themes.php (themes screen) to match that in theme.php (customizer screen). Both now read: "Close details dialog"

#4 @SergeyBiryukov
11 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 32332:

Merge two strings from theme details dialog.

props McGuive7.
fixes #32047.

Note: See TracTickets for help on using tickets.