craftcms / craftcms/ckeditor

Very poor performance when field is present multiple times.

Open
#77 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
PHP
Stars
47
Forks
32
PR merge metrics
No merged PRs in 30d

Description

We have a matrix field with 10 options that our client can use to build their content. Some of those layers make use of the new CKEditor field. While testing, I found that having more than 5 CKEditor fields in your entry takes a big performance hit on the editors browser.

Adding new fields, resizing the browser or editing content within any type of field takes multiple seconds and the fans on my MBP kick in instantly. I have attached a video below where you can see the bad performance when resizing, adding and deleting fields.. This site in the video below is not hosted locally.

https://user-images.githubusercontent.com/52166423/233072816-d8a670fa-fd42-4e54-9db7-9c79fff6557a.mp4

If you need any other information, please let me know.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the issue with more than five CKEditor fields in one matrix entry, using the attached video as the reference. Check browser behavior while adding, deleting, resizing, and editing fields; done means these actions no longer take multiple seconds or immediately cause high fan activity.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
frontend, performance
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.