fix #741

slab · Jun 20, 2016 · e5651c6 · e5651c6
1 parent e9cf8ee
commit e5651c6
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 1 deletion.
diff --git a/blots/scroll.js b/blots/scroll.js
@@ -32,7 +32,6 @@ class Scroll extends Parchment.Scroll {
     super.deleteAt(index, length);
     if (last != null && first !== last && firstOffset > 0 &&
         !(first instanceof BlockEmbed) && !(last instanceof BlockEmbed)) {
-      let lastChild = first.children.tail;
       last.moveChildren(first);
       last.remove();
     }

diff --git a/formats/list.js b/formats/list.js
@@ -18,6 +18,14 @@ class ListItem extends Block {
     }
   }
 
+  remove() {
+    if (this.prev == null && this.next == null) {
+      this.parent.remove();
+    } else {
+      super.remove();
+    }
+  }
+
   replaceWith(name, value) {
     this.parent.isolate(this.offset(this.parent), this.length());
     if (name === List.blotName) {

diff --git a/test/unit/formats/list.js b/test/unit/formats/list.js
@@ -192,6 +192,12 @@ describe('List', function() {
     expect(this.container).toEqualHTML('<ol><li>0178</li></ol>');
   });
 
+  it('delete partial', function() {
+    let editor = this.initialize(Editor, '<p>0123</p><ul><li>5678</li></ul>');
+    editor.deleteText(2, 5);
+    expect(this.container).toEqualHTML('<p>0178</p>');
+  });
+
   it('nested list replacement', function() {
     let editor = this.initialize(Editor, `
       <ol>