Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#52309 closed defect (bug) (fixed)

Contextualize "Public" website attribute translation string in Multisite settings

Reported by: audrasjb's profile audrasjb Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.7 Priority: normal
Severity: normal Version:
Component: I18N Keywords: has-patch
Focuses: ui Cc:

Description

In wp-admin/network/site-info.php, the "Public" translation string should be contextualized, especially for languages were "public" can be masculine or feminine depending on context.

$attribute_fields = array( 'public' => __( 'Public' ) );
if ( ! $is_main_site ) {
	$attribute_fields['archived'] = __( 'Archived' );
	$attribute_fields['spam']     = _x( 'Spam', 'site' );
	$attribute_fields['deleted']  = __( 'Deleted' );
}

I'd suggest to contextualize this string, just as it is for the "Spam" translation string.

Attachments (1)

52309.diff (681 bytes) - added by audrasjb 4 years ago.

Download all attachments as: .zip

Change History (4)

@audrasjb
4 years ago

#1 @audrasjb
4 years ago

  • Keywords has-patch added

#2 @SergeyBiryukov
4 years ago

  • Milestone changed from Awaiting Review to 5.7
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#3 @SergeyBiryukov
4 years ago

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

In 49962:

I18N: Add context for the Public string in site attributes list on Edit Site screen.

This allows for the string to be distinguished from a post status of the same name, which is useful for better translations in languages were "public" can be masculine or feminine depending on context.

Props audrasjb.
Fixes #52309.

Note: See TracTickets for help on using tickets.