Closed
Bug 49331
Opened 23 years ago
Closed 17 years ago
Search/Find text "Match upper/lower case" is ambiguous
Categories
(SeaMonkey :: UI Design, defect, P3)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mcoleman2, Assigned: ajschult784)
References
Details
(Whiteboard: patch-maker [good first bug])
Attachments
(1 file, 2 obsolete files)
6.40 KB,
patch
|
neil
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
From Bugzilla Helper: User-Agent: Mozilla/5.0 (X11; U; Linux 2.4.0-test1 i686; en-US; m17) Gecko/20000807 BuildID: 2000080712 The text for the the Search/Find dialog is ambiguous, to me at least. "Match upper/lower case" suggests either that upper must match upper and lower must match lower (case sensitive) or that characters typed in will match upper or lower (case insensitive). Reproducible: Always Steps to Reproduce: 1.Look 2. 3. Expected Results: Maybe just say "case sensitive?"
Updated•23 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 1•23 years ago
|
||
In the future, hopefully we will have stuff for matching diacritics and all sorts of other minor variants of characters which occur in particular writing systems. So whatever we do should be forwards-compatible if possible. I suggest `Exact case' (see bug 7930).
Comment 3•23 years ago
|
||
From bug 66696: "match upper/lower case" is ambiguous because it suggests that _either_ could be matched. a much more concise way of saying "case sensitive" would be "match case".
Comment 5•22 years ago
|
||
Chaning the qa contact on these bugs to me. MPT will be moving to the owner of this component shortly. I would like to thank him for all his hard work as he moves roles in mozilla.org...Yada, Yada, Yada...
QA Contact: mpt → zach
Comment 6•22 years ago
|
||
updating to new owner. sorry for the spam.
Assignee: hangas → mpt
Status: ASSIGNED → NEW
Comment 7•22 years ago
|
||
Suggest to move OS field from Linux to all. Furthermore I am not sure that the dependency on 7930 is true. I think the bug can be fixed without fixing the other one. Can this be checked? This seems like a trivial bug that could be quickly corrected. Today, there are only 7 references to upper/lower, as shown by this URL http://lxr.mozilla.org/seamonkey/search?string=upper%2Flower http://lxr.mozilla.org/seamonkey/source/extensions/help/resources/locale/en-US/composer_help.html#318 http://lxr.mozilla.org/seamonkey/source/extensions/help/resources/locale/en-US/composer_help.html#335 http://lxr.mozilla.org/seamonkey/source/extensions/help/resources/locale/en-US/nav_help.html#555 http://lxr.mozilla.org/seamonkey/source/xpfe/components/find/resources/locale/en-US/finddialog.dtd#6 http://lxr.mozilla.org/seamonkey/source/xpfe/components/find/resources/locale/en-US/replacedialog.dtd#6 http://lxr.mozilla.org/seamonkey/source/l10n/langpacks/en-DE/chrome/en-DE/global/locale/finddialog.dtd#5 http://lxr.mozilla.org/seamonkey/source/l10n/langpacks/en-GB/chrome/en-GB/global/locale/finddialog.dtd#5 But I guess that there may be other locales to fix and/pr that the search I've done on LXR may bot be perfect.
Updated•22 years ago
|
OS: Linux → All
Hardware: PC → All
Whiteboard: patch-maker
Comment 8•22 years ago
|
||
Changes text to "Match Exact Case" for both the find and find&replace dialogs. Only works for en-US. I think the rest is up to the localization people.
Comment 9•22 years ago
|
||
Ahh, I didn't see what you meant in the last post. This patch changes the text for all of those locations, except for the non-en-US ones. Sorry `bout that.
Attachment #59130 -
Attachment is obsolete: true
Comment 10•22 years ago
|
||
I like it! Perhaps need review? BTW what about the dependency?
Comment 11•22 years ago
|
||
I think that the dependency is sora unrelated, that this will fix this current dialog, and if they redo the dialog, they can still use this patch, I guess, I only changed DTD's and help files. If someone special could check this out i'd be very nice. There's not alot to review, no code, it's mostly just a matter of if it should be changed or not.
Comment 12•21 years ago
|
||
.
Assignee: mpt → blaker
Component: User Interface Design → XP Apps: GUI Features
QA Contact: zach → paw
Comment 13•20 years ago
|
||
This bug has existed for nearly three years now, with a year-old patch that hasn't been accepted, and this still trips me up whenever I need to search for something. What's the status on this thing?
Updated•19 years ago
|
Product: Core → Mozilla Application Suite
Comment 14•18 years ago
|
||
(In reply to comment #13) incredible. it seems so trivial.
Comment 15•18 years ago
|
||
(In reply to comment #14) > (In reply to comment #13) > > incredible. it seems so trivial. I just got confused by this too, and wanted to file a bug report, but found out the deficiency is already known since 5 years. FC4 Mozilla 1.7.10 still shows Match upper/lower case, which I interpreted as upper OR lower case. A bit of a sad story.
Comment 16•17 years ago
|
||
does a language issue need to be addressed? line numbers for patch attachment 59217 [details] [diff] [review] have changed except for finddialog.dtd patch author Colin is not reachable
Assignee: bross2 → guifeatures
Severity: normal → minor
No longer depends on: 7930
QA Contact: pawyskoczka
Whiteboard: patch-maker → patch-maker [good first bug]
Assignee | ||
Comment 17•17 years ago
|
||
toolkit find dialog just says "Match case"
Attachment #59217 -
Attachment is obsolete: true
Attachment #240821 -
Flags: superreview?(neil)
Attachment #240821 -
Flags: review?(neil)
Updated•17 years ago
|
Attachment #240821 -
Flags: superreview?(neil)
Attachment #240821 -
Flags: superreview+
Attachment #240821 -
Flags: review?(neil)
Attachment #240821 -
Flags: review+
Assignee | ||
Updated•17 years ago
|
Assignee: guifeatures → ajschult
Assignee | ||
Comment 18•17 years ago
|
||
FIXED
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•