From 77b5d7ee304a688a2eb115ce04b460b43c0dd700 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Sun, 22 May 2016 08:43:54 -0400 Subject: [PATCH] Fix priority icon(s) position --- program/js/editor.js | 31 +++++++++++++++++-------------- 1 files changed, 17 insertions(+), 14 deletions(-) diff --git a/program/js/editor.js b/program/js/editor.js index 50ba03d..1702e21 100644 --- a/program/js/editor.js +++ b/program/js/editor.js @@ -71,6 +71,9 @@ tinymce.registered_request_token = true; tinymce.util.XHR.on('beforeSend', function(e) { e.xhr.setRequestHeader('X-Roundcube-Request', rcmail.env.request_token); + // Fix missing lang parameter on addToDictionary request (#1490634) + if (e.settings && e.settings.data && /^method=addToDictionary/.test(e.settings.data) && !/&lang=/.test(e.settings.data)) + e.settings.data += '&lang=' + ref.editor.plugins.spellchecker.getLanguage(); }); } @@ -447,7 +450,6 @@ // get selected text from tinymce editor if (ed) { - ed.getWin().focus(); // correct focus in IE & Chrome if (args.selection) text = ed.selection.getContent({format: args.format}); @@ -510,10 +512,10 @@ message = message.substring(0, p) + sig + message.substring(p, message.length); cursor_pos = p - 1; } - // empty message - else if (!message) { - message = '\n\n' + sig; - cursor_pos = 0; + // empty message or new-message mode + else if (!message || !rcmail.env.compose_mode) { + cursor_pos = message.length; + message += '\n\n' + sig; } else if (rcmail.env.top_posting && !rcmail.env.sig_below) { // at cursor position @@ -547,24 +549,25 @@ // Append the signature as a div within the body if (!sigElem) { - var body = this.editor.getBody(), - doc = this.editor.getDoc(); + var body = this.editor.getBody(); - sigElem = doc.createElement('div'); - sigElem.setAttribute('id', '_rc_sig'); + sigElem = $('<div id="_rc_sig"></div>').get(0); - if (rcmail.env.top_posting && !rcmail.env.sig_below) { + // insert at start or at cursor position in top-posting mode + // (but not if the content is empty and not in new-message mode) + if (rcmail.env.top_posting && !rcmail.env.sig_below + && rcmail.env.compose_mode && (body.childNodes.length > 1 || $(body).text()) + ) { this.editor.getWin().focus(); // correct focus in IE & Chrome var node = this.editor.selection.getNode(); - // insert at start or at cursor position if found - body.insertBefore(sigElem, node.nodeName == 'BODY' ? body.firstChild : node.nextSibling); - body.insertBefore(doc.createElement('p'), sigElem); + $(sigElem).insertBefore(node.nodeName == 'BODY' ? body.firstChild : node.nextSibling); + $('<p>').append($('<br>')).insertBefore(sigElem); } else { body.appendChild(sigElem); - position_element = rcmail.env.top_posting ? body.firstChild : $(sigElem).prev(); + position_element = rcmail.env.top_posting && rcmail.env.compose_mode ? body.firstChild : $(sigElem).prev(); } } -- Gitblit v1.9.1