WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#38542 closed defect (bug) (fixed)

Twenty Seventeen: Missing theme text domain

Reported by: anilbasnet Owned by: davidakennedy
Priority: normal Milestone: 4.7
Component: Bundled Theme Version: 4.7
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Hardcoded string Top and Social should have text domain.
Now it is __( 'Top' )

Attachments (1)

38542.diff​ (679 bytes ) - added by anilbasnet 10 years ago.

Download all attachments as: .zip

Change History (6)

@anilbasnet
10 years ago

#1 @anilbasnet
10 years ago

  • Keywords has-patch added

Patch added.

#2 @swissspidy
10 years ago

  • Milestone Awaiting Review → 4.7

#3 @davidakennedy
10 years ago

Somewhat related feedback from @afercia regarding the menu label:

Hello. Minor thing, let me know if you want I open a ticket for this. I'd consider to give a more meaningful name to the "Top" menu location. "Top Menu" would be enough I guess, also consistent with "Social Links Menu"

See: ​https://wordpress.slack.com/archives/core-themes/p1477639232004365

I think this can be addressed here as well.

This ticket was mentioned in ​Slack in #core-themes by davidakennedy. ​View the logs.
10 years ago

#5 @davidakennedy
10 years ago

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

In 39001:

Twenty Seventeen: Adds missing theme text domains to starter content

  • Also, changes the top menu label so it has better context when read alone by assistive technology.

Props anilbasnet.

Fixes #38542.

Note: See TracTickets for help on using tickets.