Opened 7 years ago

Last modified 20 months ago

#7712 assigned bug

Deskbar item sorting is not locale-sensitive (easy)

Reported by: pulkomandy Owned by: nobody
Priority: normal Milestone: R1
Component: Applications/Deskbar Version: R1/Development
Keywords: gsoc2014 Cc: starsseed@…
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

It is now possible to have localized app names in deskbar and tracker.

The french name for StyledEdit is "éditeur stylé", it should be sorted somewhere around the E letter, between "Décompresseur" (Expander) and "Gestionnaire de disques" (DriveSetup). Currently, it's listed after Wonderbrush.

Use the locale kit locale-aware sorting to make this better.

Attachments (1)

0001-Fix-7712-Deskbar-item-sorting-is-not-locale-sensitiv.patch (3.3 KB) - added by 0xffea 5 years ago.
New version with global collator

Download all attachments as: .zip

Change History (10)

comment:1 Changed 7 years ago by starsseed

Cc: starsseed@… added

comment:2 Changed 5 years ago by pulkomandy

Keywords: easy removed
Summary: Deskbar item sorting is not locale-sensitiveDeskbar item sorting is not locale-sensitive (easy)

comment:3 Changed 5 years ago by 0xffea

Has a Patch: set

comment:4 Changed 5 years ago by pulkomandy

Creating a BCollator is an expansive operation (it loads data from disk to configure it to the current locale). So, it should be done only once, and kept for the whole lifetime of the TExpandoMenuBar.

Changed 5 years ago by 0xffea

New version with global collator

comment:5 Changed 5 years ago by 0xffea

A locale aware version of NaturalCompare in WindowMenuItem.cpp is also needed, but that should be a new ticket.

comment:6 Changed 5 years ago by mmadia

Keywords: gsoc2014 added

comment:7 Changed 2 years ago by jscipione

App names sorted according to locale in hrev50507

We still need to make a locale-aware version of NaturalCompare before we can fully close this ticket.

comment:8 Changed 23 months ago by pulkomandy

Has a Patch: unset

Marking patch as obsolete because it is already applied. Leaving open because of NaturalCompare as mentionned above.

Last edited 23 months ago by pulkomandy (previous) (diff)

comment:9 Changed 20 months ago by axeld

Owner: changed from axeld to nobody
Status: newassigned
Note: See TracTickets for help on using tickets.