Insert quotes at caret position; replace text selection with quotes; fix an old bug. Close #24

4chan_x.user.js

@@ -1658,15 +1658,17 @@
       text = ">>" + id + "\n";
       selection = window.getSelection();
       if (s = selection.toString()) {
-        selectionID = (_ref = $.x('preceding::input[@type="checkbox"][1]', selection.anchorNode)) != null ? _ref.name : void 0;
+        selectionID = (_ref = $.x('ancestor::blockquote', selection.anchorNode)) != null ? _ref.parentNode.firstElementChild.name : void 0;
         if (selectionID === id) {
           s = s.replace(/\n/g, '\n>');
           text += ">" + s + "\n";
         }
       }
       ta = $('textarea', qr.el);
+      ta.value = ta.value.slice(0, ta.selectionStart) + text + ta.value.slice(ta.selectionEnd, ta.value.length);
       ta.focus();
-      return ta.value += text;
+      ta.selectionEnd = ta.selectionStart + text.length;
+      return window.getSelection().collapseToEnd();
     },
     refresh: function() {
       var m, newFile, oldFile, _ref;

changelog

@@ -1,4 +1,7 @@
 master
+- mayhem
+  quotes are now inserted at the caret position in the QR
+  quotes also replace the text selection in the QR
 - aeosynth
   update the captcha caching expiration date to 30mins
 

script.coffee

@@ -1295,14 +1295,18 @@ qr =
 
     selection = window.getSelection()
     if s = selection.toString()
-      selectionID = $.x('preceding::input[@type="checkbox"][1]', selection.anchorNode)?.name
-      if selectionID == id
+      selectionID = $.x('ancestor::blockquote', selection.anchorNode)?.parentNode.firstElementChild.name
+      if selectionID is id
         s = s.replace /\n/g, '\n>'
         text += ">#{s}\n"
 
     ta = $ 'textarea', qr.el
+    #replace selection for text
+    ta.value = ta.value.slice(0, ta.selectionStart) + text + ta.value.slice(ta.selectionEnd, ta.value.length)
     ta.focus()
-    ta.value += text
+    #move the caret to the end of the new quote
+    ta.selectionEnd = ta.selectionStart + text.length
+    window.getSelection().collapseToEnd()
 
   refresh: ->
     $('[name=sub]', qr.el).value = ''