From 0c903eb890aecbe168931d45e4c358f420af6ae9 Mon Sep 17 00:00:00 2001
From: Jason Chen <jhchen7@gmail.com>
Date: Sun, 28 Aug 2016 22:40:31 -0700
Subject: [PATCH] Revert relying on clipboardData html

When pasting across applications (ex. IE11 -> Chrome) clipboardData html
contains <script> and <style> tags. The browser's native paste resolves
stripping the former and other unnecessary tags (<title>) and applying
appropriate styles to nodes. We should not reimplement all of this.

Fixes #886
---
 modules/clipboard.js | 22 ++++++----------------
 1 file changed, 6 insertions(+), 16 deletions(-)

diff --git a/modules/clipboard.js b/modules/clipboard.js
index 1fc724960c..f75fa68d74 100644
--- a/modules/clipboard.js
+++ b/modules/clipboard.js
@@ -117,26 +117,16 @@ class Clipboard extends Module {
     let range = this.quill.getSelection();
     let delta = new Delta().retain(range.index).delete(range.length);
     let types = e.clipboardData ? e.clipboardData.types : null;
-    if ((types instanceof DOMStringList && types.contains("text/html")) ||
-        (types && types.indexOf && types.indexOf('text/html') !== -1)) {
-      this.container.innerHTML = e.clipboardData.getData('text/html');
-      paste.call(this);
-      e.preventDefault();
-    } else {
-      let bodyTop = document.body.scrollTop;
-      this.container.focus();
-      setTimeout(() => {
-        paste.call(this);
-        document.body.scrollTop = bodyTop;
-        this.quill.selection.scrollIntoView();
-      }, 1);
-    }
-    function paste() {
+    let bodyTop = document.body.scrollTop;
+    this.container.focus();
+    setTimeout(() => {
       delta = delta.concat(this.convert());
       this.quill.updateContents(delta, Quill.sources.USER);
       // range.length contributes to delta.length()
       this.quill.setSelection(delta.length() - range.length, Quill.sources.SILENT);
-    }
+      document.body.scrollTop = bodyTop;
+      this.quill.selection.scrollIntoView();
+    }, 1);
   }
 }
 Clipboard.DEFAULTS = {