Issue Details (XML | Word | Printable)

Key: MBS-4614
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Normal Normal
Assignee: Ian McEwen
Reporter: nikki
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
MusicBrainz Server

Country dropdown is sorted by English name

Created: 27/Apr/12 11:10 PM   Updated: 05/Oct/12 11:41 PM   Resolved: 01/Oct/12 01:25 PM
Component/s: Internationalization
Affects Version/s: None
Fix Version/s: 2012-10-01

Issue Links:
Relates
 


 Description  « Hide

The country dropdown, such as on http://i18n.mbsandbox.org/artist/create is sorted by the English name. Once translated, that will make it rather difficult to find things in the list.

I can't check the language/script ones because of MBS-4613 but they will probably have the same problem.



Sort Order: Ascending order - Click to sort in descending order
Ian McEwen added a comment - 13/Jun/12 06:44 PM

I think this may be sorted by the ID number in the country table, and the alphabetization in English is incidental.

Still needs fixing, of course.


Ian McEwen added a comment - 06/Sep/12 01:46 AM

Oliver Charles added a comment - 10/Sep/12 02:59 PM

This failed beta testing due to MBS-5286, so I have re-opened it and moved it to the next fix version.


Ian McEwen added a comment - 10/Sep/12 08:34 PM

Mihkel Tõnnov added a comment - 05/Oct/12 01:15 PM

Script dropdown is still sorted by the English names (see e.g. http://beta.musicbrainz.org/release/add?artist=89ad4ac3-39f7-470e-963a-56509c546377).


Mihkel Tõnnov added a comment - 05/Oct/12 01:29 PM - edited

On a related note, could the language and script dropdowns be sorted case-insensitively? This would benefit the Finnish and Estonian translations (and possibly French), because language and script names in these are normally lowercase , but some begin with a name of a country (as in "Canadian Syllabics", "Inari Sami"), so would need to begin with a capital, but that shouldn't get them displayed at the beginning of the list.


Ian McEwen added a comment - 05/Oct/12 11:41 PM

Meh, forgot to do a rebase when the patch got pulled out of beta. I'm directly cherry-picking the one relevant commit.

Latter problem should probably be a new ticket