Issue #3584: improved richtext performance on large emails

the data copying seems to cause a huge chunk of the lag, so i moved this into the timeout. i also increased timeout to half a second as that should be sufficient.
RotherOSS · Sep 12, 2024 · ea51116 · ea51116
1 parent 9be6621
commit ea51116
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/var/httpd/htdocs/js/Core.UI.RichTextEditor.js b/var/httpd/htdocs/js/Core.UI.RichTextEditor.js
@@ -365,15 +365,15 @@ Core.UI.RichTextEditor = (function (TargetNS) {
 
                 //Update validation error tooltip while content is added to the editor
                 editor.model.document.on('change:data', () => {
-                    if (editor.getData() != "") {
-                        $("#" + editor.ElementId).val(editor.getData());
-                    }
-
                     window.clearTimeout(TimeOutRTEOnChange);
                     TimeOutRTEOnChange = window.setTimeout(function () {
+                        let EditorAreaContent = editor.getData();
+                        if (EditorAreaContent != "") {
+                            $("#" + editor.ElementId).val(EditorAreaContent);
+                        }
                         Core.Form.Validate.ValidateElement($EditorArea);
                         Core.App.Publish('Event.UI.RichTextEditor.ChangeValidationComplete', [editor]);
-                    }, 250);
+                    }, 500);
                 });
 
                 editor.ui.focusTracker.on('change:isFocused', (evt, name, isFocused) => {