search

GrapesJS / grapesjs

Free and Open source Web Builder Framework. Next generation tool for building templates without coding
https://grapesjs.com
BSD 3-Clause "New" or "Revised" License
22.36k stars 4.05k forks source link

BUG: When pasting text into text, the view scrolls and loses focus #4024

Closed mingxin-yang closed 2 years ago

mingxin-yang commented 2 years ago

GrapesJS version

What browser are you using?

Chrome

Reproducible demo link

https://grapesjs.com/demo-newsletter-editor.html

Describe the bug

How to reproduce the bug?

  1. https://grapesjs.com/demo-newsletter-editor.html use CKeditor, I don’t know if it’s a configuration problem
  2. First enter a long content in the text component and scroll to the middle of the screen
  3. Then when you paste a piece of text in the middle, the view will scroll, but it is not the pasted position 录屏 (1)

What is the expected behavior? ...

What is the current behavior? ..

If is necessary to execute some code in order to reproduce the bug, paste it here below:

// your code here

Code of Conduct

mingxin-yang commented 2 years ago

@artf

artf commented 2 years ago

This issue comes from CKEditor itself so, if there is a patch to apply, it should be applied on the plugin level (I'm not even sure it's possible) as it's not related to the core.

bernhardmiller commented 2 years ago

@mingxin-yang I had a similar problem with CKEditor and scrolling. For me, it happened when the user pressed the return key. If the grapes document was too long, the canvas scrolled down way to far.

I had to solve the problem in CKEditor itself, by removing this line from the enterkey plugin: https://github.com/ckeditor/ckeditor4/blob/f6dd30807a1c7cb585f376a38fb13dffd2213a75/plugins/enterkey/plugin.js#L432

Maybe there is a similar solution for your scrolling problem.