Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#10066 closed defect (bug) (fixed)

Issues with timezone picker in RC1

Reported by: xibe's profile xibe Owned by: ryan's profile ryan
Milestone: 2.8 Priority: high
Severity: major Version: 2.8
Component: I18N Keywords: has-patch
Focuses: Cc:

Description

I should have seen it earlier, but while making a quick test of the fr_FR translation with RC1, I noticed two issues in the TZ picker.

Note: the "wordpress-continents-cities.pot" has been translated (albeit partially) into "continents-cities-fr_FR.po" for a long time now, and is up to date with that latest POT.

  • some continents names are not translated, even though they are in the local po/mo. In fact (I find this out while making a list of the faulty continents), ALL of them are untranslated, SAVE for the first one: "Africa" does show up as "Afrique".
  • cities are not alphabetically sorted after translation. The fr_FR has issues which I attributed to special characters (like strings starting with 'Île' (Island) showing on top (or not!), then I stumbled upon this gem (there might be others): Douala > El Aaiun > Freetown is displayed as Douala > Laâyoune > Freetown. Please not that, following Nikolay's advices, Douala and Freetown are left untranslated in the fr_FR po/mo. In short, they are displayed with the original sorting. Dunno if the non-translations is what is breaking things.

Attached picture shows the whole Atlantic continent as displayed in 2.8-RC1 with the latest fr_FR translation. I added 'Île' to some of the entries, and this get 'Île' scattered around in the continent's sub-list.

These are not blocker items, but still, it makes for some strings being moved around in the list, which can prove untidy.

Attachments (3)

wp-28rc1-tzpicker.png (4.4 KB) - added by xibe 16 years ago.
Capture of an unsorted section of the timezone-picket in 2.8-RC1-fr_FR
add-domain-to-translate-continent.diff (545 bytes) - added by nbachiyski 16 years ago.
timezone-translate-sort.diff (1.1 KB) - added by nbachiyski 16 years ago.
Replace _ before translating names in the sort routine

Download all attachments as: .zip

Change History (10)

@xibe
16 years ago

Capture of an unsorted section of the timezone-picket in 2.8-RC1-fr_FR

#1 @nbachiyski
16 years ago

  • Keywords has-patch added; needs-patch removed

A translate() call needs a missing domain.

#2 @Denis-de-Bernardy
16 years ago

  • Priority changed from normal to high
  • Severity changed from normal to major

#3 @azaozz
16 years ago

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

(In [11533]) Add translation domain to the timezone picker, props nbachiyski, fixes #10066

#4 @xibe
16 years ago

Just curious: does this fix both issues? It seems to me it would only fix the first one - but I might be wrong, of course...

@nbachiyski
16 years ago

Replace _ before translating names in the sort routine

#5 @nbachiyski
16 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

#6 @nbachiyski
16 years ago

The sorting routine tried to translate the names before the _ was replaced.

I think the sorting is screwed enough and the patch is small/straight-forward enough in order this to get in 2.8.

#7 @automattor
16 years ago

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

(In [11544]) Replace the _ before sorting the timezones. Fixes #10066 props nbachiyski.

Note: See TracTickets for help on using tickets.