From d008e8d378549237f4aff91a416ae89c1c36b798 Mon Sep 17 00:00:00 2001 From: Hadi Nategh Date: Wed, 23 Sep 2015 11:37:44 +0000 Subject: [PATCH] Fix mail compose resize does not work always --- mail/js/app.js | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/mail/js/app.js b/mail/js/app.js index a6020be607..c7be2fbbbd 100644 --- a/mail/js/app.js +++ b/mail/js/app.js @@ -205,12 +205,13 @@ app.classes.mail = AppJS.extend( // use a wrapper on a different url to be able to use a different fpm pool et2.menuaction = 'mail_compose::ajax_send'; var that = this; + var textAreaWidget = this.et2.getWidgetById('mail_htmltext'); this.mail_isMainWindow = false; this.compose_fieldExpander_init(); this.check_sharing_filemode(); - + this.subject2title(); - + // Set autosaving interval to 2 minutes for compose message this.W_INTERVALS.push(window.setInterval(function (){ that.saveAsDraft(null, 'autosaving'); @@ -225,9 +226,14 @@ app.classes.mail = AppJS.extend( }); /*Trigger compose_resizeHandler after the CKEditor is fully loaded*/ jQuery('#mail-compose').on ('load',function() { - window.setTimeout(function(){ - that.compose_fieldExpander(); - }, 300); + if (textAreaWidget && typeof textAreaWidget.ckeditor != 'undefined') + { + textAreaWidget.ckeditor.on('instanceReady', function(){that.compose_fieldExpander();}); + } + else + { + this.compose_fieldExpander(); + } }); //Resize compose after window resize to not getting scrollbar @@ -254,7 +260,7 @@ app.classes.mail = AppJS.extend( } else { - this.et2.getWidgetById('mail_htmltext').ckeditor.on('instanceReady', function(e) { + textAreaWidget.ckeditor.on('instanceReady', function(e) { this.focus(); }); }