diff options
| author | Stephen Blott | 2018-02-04 08:57:32 +0000 |
|---|---|---|
| committer | Stephen Blott | 2018-02-04 08:57:32 +0000 |
| commit | 1d84edc0b2fce08034316c1665b6dc75d32d9ee6 (patch) | |
| tree | 6e89833d3b29a3b4023c7eebded9de6fba704a6c | |
| parent | a461d8d5ecd1aee1ae4cd28b1e0d12dc04adc152 (diff) | |
| download | vimium-1d84edc0b2fce08034316c1665b6dc75d32d9ee6.tar.bz2 | |
Remove \I (ignore case) flag for find mode.
It appears that nobody is using this (and it is broken). So this PR
removes this feature.
Fixes #2937 (kind of).
| -rw-r--r-- | content_scripts/mode_find.coffee | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/content_scripts/mode_find.coffee b/content_scripts/mode_find.coffee index f19b5db4..5090e4ee 100644 --- a/content_scripts/mode_find.coffee +++ b/content_scripts/mode_find.coffee @@ -104,7 +104,6 @@ class FindMode extends Mode # escape sequences. '\' is the escape character and needs to be escaped itself to be used as a normal # character. here we grep for the relevant escape sequences. @query.isRegex = Settings.get 'regexFindMode' - hasNoIgnoreCaseFlag = false @query.parsedQuery = @query.rawQuery.replace /(\\{1,2})([rRI]?)/g, (match, slashes, flag) => return match if flag == "" or slashes.length != 1 switch (flag) @@ -112,12 +111,10 @@ class FindMode extends Mode @query.isRegex = true when "R" @query.isRegex = false - when "I" - hasNoIgnoreCaseFlag = true "" - # default to 'smartcase' mode, unless noIgnoreCase is explicitly specified - @query.ignoreCase = !hasNoIgnoreCaseFlag && !Utils.hasUpperCase(@query.parsedQuery) + # Implement smartcase. + @query.ignoreCase = not Utils.hasUpperCase(@query.parsedQuery) regexPattern = if @query.isRegex @query.parsedQuery |