Closed
Bug 586406
Opened 16 years ago
Closed 15 years ago
Allow case sensitivity/insensitivity in find API
Categories
(Skywriter Graveyard :: Search, enhancement)
Skywriter Graveyard
Search
Tracking
(Not tracked)
RESOLVED
INVALID
0.9
People
(Reporter: glazou, Assigned: glazou)
Details
Attachments
(1 file, 1 obsolete file)
|
1.70 KB,
patch
|
Details | Diff | Splinter Review |
This is about adding a way to search for a string with a case sensitivity or
insensitivity parameter. The change should be harmless to existing users or code
clients.
| Assignee | ||
Comment 1•16 years ago
|
||
Attachment #464916 -
Flags: review?(kdangoor)
Comment 2•16 years ago
|
||
Comment on attachment 464916 [details] [diff] [review]
proposed fix
>diff -r a6966fb94150 plugins/supported/text_editor/controllers/search.js
>--- a/plugins/supported/text_editor/controllers/search.js Tue Aug 10 09:27:35 2010 -0400
>+++ b/plugins/supported/text_editor/controllers/search.js Wed Aug 11 22:21:34 2010 +0200
>@@ -123,11 +123,12 @@
> * @param isRegExp True if the text is a regex, false if it's a literal
> * string.
Please update the JavaDoc here. Rest looks good (not tested yet).
Attachment #464916 -
Flags: review?(kdangoor) → review?(jviereck)
| Assignee | ||
Comment 3•16 years ago
|
||
Attachment #464916 -
Attachment is obsolete: true
Attachment #464916 -
Flags: review?(jviereck)
Updated•16 years ago
|
Target Milestone: --- → 0.9
Comment 4•15 years ago
|
||
ACETRANSITION
The Skywriter project has merged with Ajax.org's Ace project (the full server part of which is their Cloud9 IDE project). Background on the change is here:
http://mozillalabs.com/skywriter/2011/01/18/mozilla-skywriter-has-been-merged-into-ace/
The bugs in the Skywriter product are not necessarily relevant for Ace and quite a bit of code has changed. For that reason, I'm closing all of these bugs. Problems that you have with Ace should be filed in the Ace issue tracker at GitHub:
https://github.com/ajaxorg/ace/issues
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•