Closed
Bug 1190347
Opened 10 years ago
Closed 3 years ago
Folder list in subscribe window has differing sort order than 3-pane
Categories
(Thunderbird :: Folder and Message Lists, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 231138
People
(Reporter: peter+bmo, Unassigned)
Details
Attachments
(1 file)
43.87 KB,
image/png
|
Details |
3-pane windows's sort order doesn't honor upper/lower case when sorting folders, but the (IMAP) subscribe windows does.
See screenshot in attachment.
Comment 1•8 years ago
|
||
Which order are you suggesting is correct?
Should it be case insensitive?
Flags: needinfo?(peter+bmo)
Reporter | ||
Comment 2•8 years ago
|
||
I would opt for case-insensitive.
Thanks for asking!
Flags: needinfo?(peter+bmo)
This is true and the sort also may not be locale aware. We don't have the luxury of String.localeCompare() from JS in C++ code. But maybe there is some similar mechanism.
We could probably make the sorting case insensitive straight away, there is a plain strcmp() at https://dxr.mozilla.org/comm-central/source/mailnews/base/src/nsSubscribableServer.cpp#521 and below.
Flags: needinfo?(jorgk)
This also appears to be the same as bug 231138.
Comment 5•7 years ago
|
||
Zibi, is there a C++ equivalent of String.localeCompare(). I'm quite sure there is.
Flags: needinfo?(jorgk) → needinfo?(gandalf)
Comment 6•7 years ago
|
||
Well, you can just use the ICU function: https://searchfox.org/mozilla-central/source/intl/icu/source/i18n/ucol.cpp#441
Here's example of how we do this in SpiderMonkey: https://searchfox.org/mozilla-central/source/js/src/builtin/intl/Collator.cpp#407
Here's some example of how not to do this with a response on how to do this: https://stackoverflow.com/questions/6028339/icu-and-string-compare
HTH!
Flags: needinfo?(gandalf)
Updated•3 years ago
|
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•