make quotes check more generic; add auto quote insertion for javascri…

…pt (backtick)
ajaxorg · Feb 22, 2023 · a6d9c21 · a6d9c21
1 parent f8b1ca7
commit a6d9c21
Show file tree

Hide file tree

Showing 4 changed files with 38 additions and 6 deletions.
diff --git a/src/mode/behaviour/behaviour_test.js b/src/mode/behaviour/behaviour_test.js
@@ -287,6 +287,16 @@ module.exports = {
         exec("insertstring", 1, '`');
         exec("insertstring", 1, 'b');
         assert.equal(editor.getValue(), "`b`");
+
+        editor.setValue("");
+        exec("insertstring", 1, 'a');
+        exec("insertstring", 1, "'");
+        assert.equal(editor.getValue(), "a'");
+
+        editor.setValue("");
+        exec("insertstring", 1, 'a');
+        exec("insertstring", 1, "`");
+        assert.equal(editor.getValue(), "a``");
     },
     "test: css": function() {
         editor.session.setMode(new CSSMode());

diff --git a/src/mode/behaviour/cstyle.js b/src/mode/behaviour/cstyle.js
@@ -261,9 +261,17 @@ var CstyleBehaviour = function(options) {
                     var isWordBefore = wordRe.test(leftChar);
                     wordRe.lastIndex = 0;
                     var isWordAfter = wordRe.test(rightChar);
-                    let hasStringPrefixes = (options && options.quotePrefixes &&
-                        Array.isArray(options.quotePrefixes)) ? options.quotePrefixes.includes(leftChar) : null;
-
+                    var hasStringPrefixes = false;
+                    if (options && options.quotesPrefixes && Array.isArray(options.quotesPrefixes)) {
+                        for (var prefix of options.quotesPrefixes) {
+                            if (prefix.quotes && prefix.condition && prefix.quotes instanceof RegExp
+                                && prefix.quotes.test(quotes[text]) && prefix.condition instanceof RegExp
+                                && prefix.condition.test(leftChar)) {
+                                hasStringPrefixes = true;
+                                break;
+                            }
+                        }
+                    }
                     if ((!hasStringPrefixes && isWordBefore) || isWordAfter)
                         return null; // before or after alphanumeric
                     if (rightChar && !/[\s;,.})\]\\]/.test(rightChar))

diff --git a/src/mode/javascript.js b/src/mode/javascript.js
@@ -10,9 +10,16 @@ var CStyleFoldMode = require("./folding/cstyle").FoldMode;
 
 var Mode = function() {
     this.HighlightRules = JavaScriptHighlightRules;
-    
+
     this.$outdent = new MatchingBraceOutdent();
-    this.$behaviour = new CstyleBehaviour();
+    this.$behaviour = new CstyleBehaviour({
+        quotesPrefixes: [
+            {
+                quotes: new RegExp("`"),
+                condition: new RegExp("\\w")
+            }
+        ]
+    });
     this.foldingRules = new CStyleFoldMode();
 };
 oop.inherits(Mode, TextMode);

diff --git a/src/mode/python.js b/src/mode/python.js
@@ -10,7 +10,14 @@ var Range = require("../range").Range;
 var Mode = function() {
     this.HighlightRules = PythonHighlightRules;
     this.foldingRules = new PythonFoldMode("\\:");
-    this.$behaviour = new CstyleBehaviour({quotePrefixes: ["f", "F", "u", "U", "r", "R"]});
+    this.$behaviour = new CstyleBehaviour({
+        quotesPrefixes: [
+            {
+                quotes: new RegExp("['\"]"),
+                condition: new RegExp("[ufr]", "i")
+            }
+        ]
+    });
 };
 oop.inherits(Mode, TextMode);