Fix redundant blank lines when using HTML and top posting (#1490576)

Aleksander Machniak

2015-10-28 72be74508f8e8d39150fa58c1572e41a7791c02f

 program/js/editor.js

@@ -527,20 +527,19 @@

      // 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)
        if (rcmail.env.top_posting && !rcmail.env.sig_below && (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);
@@ -548,9 +547,7 @@
        }
      }

      if (rcmail.env.signatures[id]) {
        sigElem.innerHTML = rcmail.env.signatures[id].html;
      }
      sigElem.innerHTML = rcmail.env.signatures[id] ? rcmail.env.signatures[id].html : '';
    }
    else if (!rcmail.env.top_posting) {
      position_element = $(this.editor.getBody()).children().last();