Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 11 years ago

Closed 11 years ago

#31776 closed defect (bug) (fixed)

I18n optimizations for 4.2

Reported by:	obenland	Owned by:	ocean90
Priority:	high	Milestone:	4.2
Component:	I18N	Version:	4.2
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:

Description

Ticket to collect optimization suggestions for new strings in 4.2.

Attachments (7)

31776.diff (834 bytes ) - added by obenland 11 years ago.: Uppercase D to match themes.php and class-wp-customize-control.php
31776.1.diff (2.4 KB ) - added by obenland 11 years ago.: Optional: Use a placeholder to be able to reorder words.
31776.2.diff (910 bytes ) - added by obenland 11 years ago.: Add seconds in date format string for parity.
31776.4.diff (604 bytes ) - added by obenland 11 years ago.: Context for ID placeholder
31776.5.diff (2.6 KB ) - added by obenland 11 years ago.: Follow-up to 31776.1.diff
31776.3.diff (669 bytes ) - added by obenland 11 years ago.: Context for placeholder
31776.patch (1.6 KB ) - added by iseulde 11 years ago.

Download all attachments as: .zip

Change History (16)

@obenland
11 years ago

Attachment 31776.diff added

Uppercase D to match themes.php and class-wp-customize-control.php

@obenland
11 years ago

Attachment 31776.1.diff added

Optional: Use a placeholder to be able to reorder words.

@obenland
11 years ago

Attachment 31776.2.diff added

Add seconds in date format string for parity.

#1 @ocean90
11 years ago

Owner set to ocean90
Status new → accepted

@obenland
11 years ago

Attachment 31776.4.diff added

Context for ID placeholder

@obenland
11 years ago

Attachment 31776.5.diff added

Follow-up to 31776.1.diff

@obenland
11 years ago

Attachment 31776.3.diff added

Context for placeholder

#2 @ocean90
11 years ago

Improve newly added strings for i18n:

Use a placeholder for the theme name to be able to reorder words.
Uppercase D for "Theme Details" to match existing strings.
Merge two revision date formats.
Add translator comment to strings with placeholders.

props obenland.
see #31776.

#3 @ocean90
11 years ago

Improve newly added strings for i18n:

Add context to "Format" string.
Use "Format" with context for "Post format".

This ticket was mentioned in Slack in #core by drew. View the logs.

11 years ago

#5 @ocean90
11 years ago

Remove a stray > introduced in [31905].

#6 @DrewAPicture
11 years ago

Priority normal → high

@iseulde
11 years ago

Attachment 31776.patch added

#7 @iseulde
11 years ago

Keywords has-patch added

31776.patch: Merge strings that describe the same command.

This ticket was mentioned in Slack in #core by drew. View the logs.

11 years ago

#9 @ocean90
11 years ago

Resolution → fixed
Status accepted → closed

Merge strings that describe the same command.

props iseulde.
fixes #31776.

Note: See TracTickets for help on using tickets.

Download in other formats: