From 9799dd294dfb201f2f256151120d38580e34e0ec Mon Sep 17 00:00:00 2001 From: Nicolas Lempereur Date: Wed, 7 Nov 2018 13:26:56 +0000 Subject: [FIX] mail: enable composer after message sent c5dc0b2f39 disabled send button on composer sent. But this is only needed for chatter composer, other composers directly send the message and are not refreshed asynchronously. So for other composer, the send button worked one time but then was disabled and the only way to sent was ENTER (on basic composer) or CTRL+ENTER (on extended composer). With this fix, disabling the button is done only for chatter composer. Also the behavior of not clearing the message if there was an error is introduced back. Without change in code, added test fails with: "Should keep unsent message in the composer on failure" (result: "") "Send button should be re-enabled on message post failure" (result: true) issue found when checking opw-1904029 closes #28475 --- addons/mail/static/src/js/composers/basic_composer.js | 1 - 1 file changed, 1 deletion(-) (limited to 'addons/mail/static/src/js/composers/basic_composer.js') diff --git a/addons/mail/static/src/js/composers/basic_composer.js b/addons/mail/static/src/js/composers/basic_composer.js index 0e4d62232e2..a6cdb39e2de 100644 --- a/addons/mail/static/src/js/composers/basic_composer.js +++ b/addons/mail/static/src/js/composers/basic_composer.js @@ -390,7 +390,6 @@ var BasicComposer = Widget.extend({ return; } - this.$(".o_composer_button_send").prop("disabled", true); clearTimeout(this._cannedTimeout); var self = this; this._preprocessMessage().then(function (message) { -- cgit v1.2.3