Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Large text hanging the page #1374

Closed
dkrahn opened this issue Nov 26, 2018 · 3 comments
Closed

Large text hanging the page #1374

dkrahn opened this issue Nov 26, 2018 · 3 comments
Labels
resolution:duplicate This issue is a duplicate of another issue and was merged into it.

Comments

@dkrahn
Copy link
Contributor

dkrahn commented Nov 26, 2018

Is this a bug report or feature request? (choose one)

🐞 Bug report | Other

💻 Version of CKEditor

From latest examples(https://ckeditor.com/ckeditor-5/#document)
Compared to CKEditor4 lastest examples (https://ckeditor.com/ckeditor-4/#document)

📋 Steps to reproduce

  1. Have some huge text (Lorem ipsum >100 paragraphs)
  2. Paste them in the editor
  3. Select all the text
  4. Apply some style(Ex. Bold)

✅ Expected result

Have CKEditor5 apply it as quick as CKEditor4.

❎ Actual result

CKEditor4 applies it instantly, while CKEditor5 takes a while, hanging the page.

📃 Other details that might be useful

I know CKEditor5 has a lot more constrains reflecting the changes into the model. But is seams that this could impact some cases with where the documents are big. For this reason I was split between bug and other, for it seams to be JavaScript performance limitation.

@oleq
Copy link
Member

oleq commented Nov 26, 2018

@Reinmar
Copy link
Member

Reinmar commented Nov 26, 2018

We didn't make any research on this yet so I guess there are some low hanging fruits out there. Let's keep the discussion in https://github.com/ckeditor/ckeditor5-engine/issues/494

@Reinmar Reinmar closed this as completed Nov 26, 2018
@Reinmar Reinmar added the resolution:duplicate This issue is a duplicate of another issue and was merged into it. label Nov 26, 2018
@Reinmar
Copy link
Member

Reinmar commented Nov 26, 2018

PS. Some initial profiling may help to understand what kind of issues we're dealing with. If you're struggling with performance issues in real use cases, you may try to dig into this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
resolution:duplicate This issue is a duplicate of another issue and was merged into it.
Projects
None yet
Development

No branches or pull requests

3 participants