Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#33429 closed defect (bug) (fixed)

Fix Period Position in wp-admin/about.php

Reported by: extendwings's profile extendwings Owned by: chriscct7's profile chriscct7
Milestone: 4.3.1 Priority: normal
Severity: normal Version: 4.3
Component: Text Changes Keywords: needs-refresh dev-feedback
Focuses: administration Cc:

Description

wp-admin/about.php#L89 is hard to translate. (at least, in Japanese)

Attachments (3)

33429.diff (667 bytes) - added by extendwings 10 years ago.
33429.patch (724 bytes) - added by chriscct7 10 years ago.
Patch from proper directory
33429.2.diff (2.4 KB) - added by dd32 10 years ago.

Download all attachments as: .zip

Change History (13)

@extendwings
10 years ago

@chriscct7
10 years ago

Patch from proper directory

#1 @chriscct7
10 years ago

  • Component changed from Help/About to Text Changes
  • Focuses administration added
  • Keywords has-patch commit added
  • Milestone changed from Awaiting Review to 4.3.1
  • Owner set to chriscct7
  • Status changed from new to accepted

Refreshed the patch to go from the proper directory.

#2 follow-up: @dd32
10 years ago

In general, we don't like to add new strings in point releases, and this string is going to change in 4.4.
I'm quite tempted to suggest we simply remove the period from the placeholder and skip the period in the string, or, simply suffixing the period outside of the translation.

33429.patch isn't the same as 33429.diff - the latter moves the period from the placeholder to the string, the former misses removing it from the placeholder.

#3 @chriscct7
10 years ago

  • Keywords needs-refresh added; has-patch commit removed

Indeed it seems the period is missing. I'll get that refreshed

@dd32
10 years ago

#4 in reply to: ↑ 2 ; follow-up: @dd32
10 years ago

Replying to dd32:

I'm quite tempted to suggest we simply remove the period from the placeholder and skip the period in the string, or, simply suffixing the period outside of the translation.

33429.2.diff implements this, no string change, and suffixes the period even if the placeholders have changed in their orders.

This assumes all languages would have a period to end a sentence.
Looking at existing translations, the following do NOT use a ending period (or it's translation errors)

  • Thai, Armenian, Japanese, Bengali, Chinese (Taiwan), Dzongkha

The following DO use one:

  • Swedish, English (Australia), English (Canada), Hebrew, Romanian, German, German (Formal), Finnish, Dutch, Welsh, Greek, Polish, French, Italian, Catalan, Basque, Indonesian, Spanish, Galician, Bosnian, Esperanto, Slovak, Arabic, Korean, Turkish, Serbian, Norwegian (Bokmål), Norwegian (Nynorsk), Bulgarian, Lithuanian, Croatian, English (UK), English (New Zealand), Icelandic, Brazilian Portuguese, Hungarian, Portuguese, Albanian, Spanish (Mexico), Danish, Macedonian, Ukrainian, Russian, Peruvian Spanish

@ocean90 What's your thoughts here? Should we just change the string?

#5 in reply to: ↑ 4 @ocean90
10 years ago

Replying to dd32:

@ocean90 What's your thoughts here? Should we just change the string?

The about page is one of the rarely visited screens. I wouldn't change it if it's the only string change.

#6 @chriscct7
10 years ago

  • Keywords dev-feedback added

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


10 years ago

#8 @SergeyBiryukov
10 years ago

There'll be a version bump in about.php for 4.3.1, so might as well fix this (without a string change).

#9 @ocean90
10 years ago

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

In 34054:

About: Remove period from placeholder.

The position of periods is different in some languages, like Japanese. This removes the period from the last placeholder for the Template Hierarchy section. It doesn't include a new period to avoid a string change in a minor release.

Props extendwings, chriscct7.
Fixes #33429.

#10 @ocean90
10 years ago

In 34055:

About: Remove period from placeholder.

The position of periods is different in some languages, like Japanese. This removes the period from the last placeholder for the Template Hierarchy section. It doesn't include a new period to avoid a string change in a minor release.

Merge of [34054] to the 4.3 branch.

Props extendwings, chriscct7.
See #33429.

Note: See TracTickets for help on using tickets.