IE11 performance when pasting large content is terrible #1977

Closed
avif opened this Issue Jul 10, 2016 · 0 comments

Projects

None yet

1 participant

@avif
Contributor
avif commented Jul 10, 2016

steps to reproduce

  1. Copy large content.
  2. Paste it in a summernote editor.
  3. Go make a cup of coffee.

browser version and os version

Windows 8.1, IE11 (11.0.9600.18350)

  • I'm using a VM but it happens on a dedicated Windows box too.

JSBin

http://jsbin.com/widulopaqi/edit?html,js,output
try copy and pasting the entire wikipedia page for JS for example:
https://en.wikipedia.org/wiki/JavaScript

Possible root cause

Editor module, IE workaround event handler for "change" event is called to many times, line 76 - 81:

// [workaround] IE doesn't have input events for contentEditable
      // - see: https://goo.gl/4bfIvA
      var changeEventName = agent.isMSIE ? 'DOMCharacterDataModified DOMSubtreeModified DOMNodeInserted' : 'input';
      $editable.on(changeEventName, function () {
        context.triggerEvent('change', $editable.html());
      });

Possible fix

Debounce the event handler! I'll add a PR later.

@avif avif added a commit to avif/summernote that referenced this issue Jul 10, 2016
@avif avif Fixed #1977. 669f35b
@avif avif referenced this issue Jul 10, 2016
Merged

Fixed #1977. #1978

@hackerwins hackerwins closed this in #1978 Aug 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment