Replace 'ignorecase' and 'smartcase' with 'searchcase'.

common/content/dactyl.js

@@ -686,7 +686,7 @@ const Dactyl = Module("dactyl", {
         let res = <res>
                 <dt>{link(obj.name)}</dt> <dd>{obj.description ? obj.description.replace(/\.$/, "") : ""}</dd>
             <item>
-                <tags>{template.map(obj.names.reverse, tag, " ")}</tags>
+                <tags>{template.map(obj.names.slice().reverse(), tag, " ")}</tags>
                 <spec>{spec((obj.specs || obj.names)[0])}</spec>{
                 !obj.type ? "" : <>
                 <type>{obj.type}</type>
@@ -721,7 +721,7 @@ const Dactyl = Module("dactyl", {
                                 })</>
                         }</>
                     ]));
-        return res.*.toXMLString().replace(/^ {12}/gm, "");;
+        return res.*.toXMLString().replace(/^ {12}|[ \t]+$/gm, "");
     },
 
     /**

common/content/events.js

@@ -969,11 +969,9 @@ const Events = Module("events", {
                 this._input.motionCount = null;
 
                 if (!isEscape(key)) {
-                    stop = false;
-                    if (mode.main === modes.TEXT_EDIT) {
+                    stop = (mode.main === modes.TEXT_EDIT);
+                    if (stop)
                         dactyl.beep();
-                        stop = true;
-                    }
 
                     if (mode.main === modes.COMMAND_LINE)
                         if (!(mode.extended & modes.INPUT_MULTILINE))

common/content/finder.js

@@ -28,7 +28,8 @@ const RangeFinder = Module("rangefinder", {
         let highlighted = this.rangeFind && this.rangeFind.highlighted;
         let selections = this.rangeFind && this.rangeFind.selections;
         let regexp = false;
-        let matchCase = options["smartcase"] && /[A-Z]/.test(str) || !options["ignorecase"];
+        let matchCase = options["searchcase"] === "smart"  ? /[A-Z]/.test(str) :
+                        options["searchcase"] === "ignore" ? false : true;
         let linksOnly = options["linksearch"];
 
         str = str.replace(/\\(.|$)/g, function (m, n1) {
@@ -218,9 +219,16 @@ const RangeFinder = Module("rangefinder", {
                 }
             });
 
-        options.add(["ignorecase", "ic"],
-            "Ignore case in search patterns",
-            "boolean", true);
+        options.add(["searchcase", "sc"],
+            "Search case matching mode",
+            "string", "smart",
+            {
+                completer: function () [
+                    ["smart", "Case is significant when capital letters are typed"],
+                    ["match", "Case is always significant"],
+                    ["ignore", "Case is never significant"]
+                ]
+            });
 
         options.add(["incsearch", "is"],
             "Show where the search pattern matches as it is typed",
@@ -230,9 +238,6 @@ const RangeFinder = Module("rangefinder", {
             "Limit the search to hyperlink text",
             "boolean", false);
 
-        options.add(["smartcase", "scs"],
-            "Override the 'ignorecase' option if the pattern contains uppercase characters",
-            "boolean", true);
     }
 });
 

common/locale/en-US/index.xml

@@ -392,7 +392,6 @@ This file contains a list of all available commands, mappings and options.
     <dt><o>hinttimeout</o></dt> <dd>Timeout before automatically following a non-unique numerical hint</dd>
     <dt><o>history</o></dt> <dd>Number of Ex commands and search patterns to store in the command-line history</dd>
     <dt><o>hlsearch</o></dt> <dd>Highlight previous search pattern matches</dd>
-    <dt><o>ignorecase</o></dt> <dd>Ignore case in search patterns</dd>
     <dt><o>incsearch</o></dt> <dd>Show where the search pattern matches as it is typed</dd>
     <dt><o>insertmode</o></dt> <dd>Use Insert mode as the default for text areas</dd>
     <dt><o>jsdebugger</o></dt> <dd>Use the JavaScript debugger service for JavaScript completion</dd>
@@ -416,12 +415,12 @@ This file contains a list of all available commands, mappings and options.
     <dt><o>sanitizeshutdown</o></dt> <dd>The items to sanitize automatically at shutdown</dd>
     <dt><o>sanitizetimespan</o></dt> <dd>The default sanitizer time span</dd>
     <dt><o>scroll</o></dt> <dd>Number of lines to scroll with <k name="C-u"/> and <k name="C-d"/> commands</dd>
+    <dt><o>searchcase</o></dt> <dd>Search case matching mode</dd>
     <dt><o>shell</o></dt> <dd>Shell to use for executing <ex>:!</ex> and <ex>:run</ex> commands</dd>
     <dt><o>shellcmdflag</o></dt> <dd>Flag passed to shell when executing <ex>:!</ex> and <ex>:run</ex> commands</dd>
     <dt><o>showmode</o></dt> <dd>Show the current mode in the command line</dd>
     <dt><o>showstatuslinks</o></dt> <dd>Show the destination of the link under the cursor in the status bar</dd>
     <dt><o>showtabline</o></dt> <dd>Control when to show the tab bar of opened web pages</dd>
-    <dt><o>smartcase</o></dt> <dd>Override the <o>ignorecase</o> option if the pattern contains uppercase characters</dd>
     <dt><o>strictfocus</o></dt> <dd>Prevent scripts from focusing input elements without user intervention</dd>
     <dt><o>suggestengines</o></dt> <dd>Engine Alias which has a feature of suggest</dd>
     <dt><o>titlestring</o></dt> <dd>Change the title of the window</dd>

common/locale/en-US/options.xml

@@ -869,17 +869,6 @@
     </description>
 </item>
 
-<item>
-    <tags>'noic' 'noignorecase'</tags>
-    <tags>'ic' 'ignorecase'</tags>
-    <spec>'ignorecase' 'ic'</spec>
-    <type>boolean</type>
-    <default>on</default>
-    <description>
-        <p>Ignore case in search patterns.</p>
-    </description>
-</item>
-
 <item>
     <tags>'nois' 'noincsearch'</tags>
     <tags>'is' 'incsearch'</tags>
@@ -1282,6 +1271,22 @@
     </description>
 </item>
 
+<item>
+    <tags>'sc' 'searchcase'</tags>
+    <spec>'searchcase'</spec>
+    <type>string</type>
+    <default>smart</default>
+    <description>
+        <p>Search case matching mode.</p>
+
+        <dl>
+            <dt>ignore</dt> <dd>Case is never significant</dd>
+            <dt>match</dt> <dd>Case is always significant</dd>
+            <dt>smart</dt> <dd>Case is significant when capital letters are typed</dd>
+        </dl>
+    </description>
+</item>
+
 <item>
     <tags>'shell' 'sh'</tags>
     <spec>'shell' 'sh'</spec>
@@ -1354,21 +1359,6 @@
     </description>
 </item>
 
-<item>
-    <tags>'noscs' 'nosmartcase'</tags>
-    <tags>'scs' 'smartcase'</tags>
-    <spec>'smartcase' 'scs'</spec>
-    <type>boolean</type>
-    <default>on</default>
-    <description>
-        <p>
-            Override the <o>ignorecase</o> option if the pattern contains
-            uppercase characters.  This is only used if the <o>ignorecase</o>
-            option is set.
-        </p>
-    </description>
-</item>
-
 <item>
     <tags>'nosf' 'nostrictfocus'</tags>
     <tags>'sf' 'strictfocus'</tags>

pentadactyl/NEWS

@@ -52,6 +52,7 @@
     * IMPORTANT: 'laststatus' has been replaced with the "s" flag in
       'guioptions'.
     * IMPORTANT: 'showstatuslinks' is now a string option.
+    * IMPORTANT: 'ignorecase' and 'smartcase' have been replaced with 'searchcase'
     * IMPORTANT: Command script files now use the *.penta file extension.
     * IMPORTANT: Plugins are now loaded from the 'plugins/'
       directory in 'runtimepath' rather than 'plugin/'.

pentadactyl/TODO

@@ -39,7 +39,6 @@ FEATURES:
 8 <C-o>/<C-i> should work as in Vim (i.e., save page positions as well as
   locations in the history list).
 8 jump to the next heading with ]h, next image ]i, previous textbox [t and so on
-7 add source location links to the output of certain commands (e.g. :viusage! or describe-key)
 7 add search capability to MOW
 7 describe-key command (prompt for a key and display its binding with documentation)
 7 Specify all INSERT mode mappings rather than falling through to the host apps