handle negative range

BryceMindLab · Jul 4, 2018 · f30d524 · f30d524
1 parent 8dfdc7c
commit f30d524
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/core/quill.js b/core/quill.js
@@ -398,7 +398,7 @@ class Quill {
       this.selection.setRange(null, length || Quill.sources.API);
     } else {
       [index, length, , source] = overload(index, length, source);
-      this.selection.setRange(new Range(index, length), source);
+      this.selection.setRange(new Range(Math.max(0, index), length), source);
       if (source !== Emitter.sources.SILENT) {
         this.selection.scrollIntoView(this.scrollingContainer);
       }

diff --git a/modules/keyboard.js b/modules/keyboard.js
@@ -350,7 +350,7 @@ Keyboard.DEFAULTS = {
         const { event, line: cell } = context;
         const offset = cell.offset(this.quill.scroll);
         if (event.shiftKey) {
-          this.quill.setSelection(Math.max(0, offset - 1), Quill.sources.USER);
+          this.quill.setSelection(offset - 1, Quill.sources.USER);
         } else {
           this.quill.setSelection(offset + cell.length(), Quill.sources.USER);
         }