#14231 closed task (fixed)
Performance issues with the Dojo Editor in IE8/7
Reported by: | Herman Lo | Owned by: | bill |
---|---|---|---|
Priority: | high | Milestone: | 1.8 |
Component: | Editor | Version: | 1.6.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Machine Spec: Intel Core Duo 2.2ghz, 3gb ram, Windows XP.
Under IE8, a dojo editor loaded with 3000 lines takes approximately 10˜ seconds to complete an operation. Attached a sample editor with the standard plugins plus three sample rudimentary plugins to demonstrate the issue (Move attached files to \dojo\dijit\tests\editor\).
this.editor. get('value'); this.editor.set('value', html); this.editor.execCommand("insert html", "<span class=\"cursorSpan\">Insert</span&g t;");
Given the performance issues with IE in general, some performance degradation is to be expected in comparison to other browsers but the performance gap is markedly noticeable. We can partially compensate for the issue by minimizing our manipulation of the text content and relying on progress dialogs but in certain cases where we need to manipulate the text in near real-time this may not be an option.
In a similar vein, we're experiencing issues with the editor after each edit. From my understanding, the editor will copy the text content into an internal array, so that it can maintain an undo history. However, in the case of the editor described above, the editor and browser will freeze for several seconds after each edit while the undo array is been populated.
Setting the customUndo flag to false, will disable the editor's undo functionality and instead fall back to the browser's native implementation of undo. However, the editor continues tracking the undo history after each edit. Is this the expected behavior?
Attachments (6)
Change History (11)
Changed 9 years ago by
Attachment: | test_Editor_performance.html added |
---|
Changed 9 years ago by
Attachment: | editor_html.patch added |
---|
patch to use cloneNode() and avoid apparently unnecessary node.outerHTML reference
comment:1 Changed 9 years ago by
Owner: | set to bill |
---|---|
Resolution: | → fixed |
Status: | new → closed |
In [26975]:
performance test