363435 - bookmark keywords are case sensitive

Status: RESOLVED FIXED

(Camino Graveyard :: Bookmarks, defect)

Description

[timeless]

product parity, firefox, seamonkey, mozilla, none of them are. just camino.

steps: have a 'bug' keyword that loads bugzilla bugs.

try using Bug 10000, it doesn't work in camino, but works everywhere else.

Comment 1

[froodian (Ian Leue)]

Attachment: Patch

The nil check is because caseInsensitiveCompare doesn't like nil arguments.

Comment 2

[Chris Lawson (gone)]

Erm, this is arguably a useful feature.

Not that I use it, but I can see how it would be useful.

cl

Comment 3

[froodian (Ian Leue)]

If it's a question of feature-parity, I'm not sure we can afford to sacrifice "works the same way" for "arguably useful."

Comment 4

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Given that nothing else in an HFS world is case-sensitive, this seems odd ;)  It will also cause problems for people importing bookmarks when migrating from Firefox....

Comment 5

[Sean Murphy]

Comment on attachment 248249 [details] [diff] [review]
Patch

r=me for the patch itself, since it looks great and works without any trouble.

I'll leave it up to the sr and other opinions as to whether we should actually institute a change in how keywords are resolved.  I am only worried if some (probably very few if any) users were taking advantage of the current behavior and storing multiple keywords differing only by case.  Anyone is this situation will now experience bug 357214, which describes what happens when two or more bookmarks have same keyword.

My opinion: Case-insensitive keywords will most likely help far more users than the ones we'd anger (the ones who are currently using identical-except-for-case keywords).  Since we strive for excellent usability and a simple design, case insensitivity seems like to way to go here.

As a side note, adding a keyword manager (see bug 201723 comment 20 - which would be easy after bug 352250 is in place) could be a nice accompaniment to this and help those crazy people taking advantage of case-sensitive keywords, who now need to change several at once.

Comment 6

[Stuart Morgan]

Case sensitivity as a feature here seems very fringe; I think we want this change.

Comment 7

[froodian (Ian Leue)]

Comment on attachment 248249 [details] [diff] [review]
Patch

a=pink, targeting sr to smorgan.

Comment 8

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 248249 [details] [diff] [review]
Patch

sr=pink

Comment 9

[froodian (Ian Leue)]

Checked in on 1.8branch and trunk.

Comment 10

[timeless]

I'll verify this later. in case it isn't obvious, I'm usually a power user- your edge case is edgier than I am.