Opened 16 years ago
Closed 16 years ago
#2999 closed Bug (fixed)
Bad performance on selection change checks
Reported by: | Frederico Caldeira Knabben | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Must have (possibly next milestone) | Milestone: | CKEditor 3.0 |
Component: | General | Version: | |
Keywords: | Confirmed | Cc: |
Description
There is a really bad performance in the tasks being fired when the selection changes in the editor. To check it:
- Clean load a sample.
- With a mouse click, place the cursor inside any unformatted word, like "are".
- Open the Firebug console and start a Profile job. Don't do anything in the editor, not even a mouseover in the toolbar buttons.
- Click inside a bold word, like "sample". Again, don't do anything else in the editor.
- Stop the Profile job.
The profiler will show you an excessive number of calls.
In IE6 this bad performance is terribly noticeable, compromising the editor usage.
Performance excellence is definitely important for this feature.
Change History (3)
comment:1 Changed 16 years ago by
comment:2 Changed 16 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | new → assigned |
The patch in #2927 could almost resolve the bug on this ticket.
comment:3 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
The situation is much more stable now. There may be room for improvements, but this is not anymore a critical issue.
This is related to #2927, which raise the state computation to O(n2), where n stand for number of commands.