Closed Bug 418251 Opened 16 years ago Closed 16 years ago

TaskMode: Categories toolbar-button popup menu are not sorted locally dependent

Categories

(Calendar :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: berend.cornelius09, Assigned: berend.cornelius09)

References

Details

(Keywords: regression)

Attachments

(1 file, 2 obsolete files)

STEPS TO REPRODUCE:
===================

- add two categories in the preference/categories dialog. One of the categories should contain a locale - specifice special character: E.G. for germany:"Z-Category" and "Ü-Category"
- Switch to task-mode and select a task
- Open the menu in the category-toolbar-button.

RESULT:
=======
The category with the special character is not correctly sorted in and will turn up in the end of the list.

EXPECTED RESULT:
================

the category with the special character should be correctly sorted in.

REPRODUCIBLE:
=============

- always
Flags: blocking-calendar0.8?
Attached patch patch v. #1 (obsolete) — Splinter Review
This is a followup bug of bug 327780 Need to sort out what sort of toolbar UI to offer for Lightning, where I partly overwrote functioality from patch bug 321010.
Attachment #304042 - Flags: review?
Attached patch patch v. #1 (obsolete) — Splinter Review
This is a followup bug of bug 327780 Need to sort out what sort of toolbar UI to offer for Lightning, where I partly overwrote functioality from patch bug 321010.
Attachment #304043 - Flags: review?(philipp)
Comment on attachment 304042 [details] [diff] [review]
patch v. #1

(again) attached patch twice...
Attachment #304042 - Flags: review?
Berend, I think you can get rid of |var categoriesString = getLocalizedPref("calendar.categories.names", "");|, and adjust the comment about the empty string and split.
Attachment #304042 - Attachment is obsolete: true
You should also replace following in setCategory(...):

setItemProperty(aItem, "CATEGORIES", category);

with

setItemProperty(item, "CATEGORIES", categoriesArrayToString([category]));
I don't think that this really blocks the release. We will take the patch of course if it arrives in time.
Status: NEW → ASSIGNED
Flags: blocking-calendar0.8? → blocking-calendar0.8-
This regressed Bug 321010 according to Bug 327780 Comment #44.
Blocks: 321010
Keywords: regression
Attached patch patch v. #2Splinter Review
mschroeder, thank you for your hints.
Attachment #304043 - Attachment is obsolete: true
Attachment #304215 - Flags: review?(philipp)
Attachment #304043 - Flags: review?(philipp)
Comment on attachment 304215 [details] [diff] [review]
patch v. #2

Looks good, r=philipp
Attachment #304215 - Flags: review?(philipp) → review+
patch v. #2 checked in on trunk and MOZILLA_1_8_BRANCH
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.8
checked in nightly build 2008022020 -> task is fixed and verified.
Status: RESOLVED → VERIFIED
Flags: blocking-calendar0.8-
You need to log in before you can comment on or make changes to this bug.