quill icon indicating copy to clipboard operation
quill copied to clipboard

Chrome unsupported DOMNodeInserted

Open lurryxiao opened this issue 1 year ago • 2 comments

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch [email protected] for the project I'm working on.

Here is the diff that solved my problem:

diff --git a/node_modules/quill/dist/quill.js b/node_modules/quill/dist/quill.js
index 811b3d0..d6b6cb4 100644
--- a/node_modules/quill/dist/quill.js
+++ b/node_modules/quill/dist/quill.js
@@ -4234,7 +4234,7 @@ var Scroll = function (_Parchment$Scroll) {
       }, {});
     }
     // Some reason fixes composition issues with character languages in Windows/Chrome, Safari
-    _this.domNode.addEventListener('DOMNodeInserted', function () {});
+    // _this.domNode.addEventListener('DOMNodeInserted', function () {});
     _this.optimize();
     _this.enable();
     return _this;

This issue body was partially generated by patch-package.

lurryxiao avatar Aug 09 '24 07:08 lurryxiao

Any chance to merge this change @luin ? it seems the listener is doing nothing at all right?

ramarivera avatar Feb 08 '25 11:02 ramarivera

This was apparently removed in v2: https://github.com/slab/quill/commit/41a60fbf7cc9d23f4d87c4a88c42bb56157e3432

robbytx avatar Apr 30 '25 16:04 robbytx