Thanks for this fix, just what I was looking for!
// add the new signature string if (this.env.signatures && this.env.signatures[id]) { sig = this.env.signatures[id]['text']; if (this.env.signatures[id]['is_html']) { sig = this.env.signatures[id]['plain_text']; } if (sig.indexOf('-- ')!=0) sig = '-- \n'+sig; message += '\n'+sig; } }
Seems like this is a question of taste. I personally also prefer the signature above the quoted text...this makes it easier to read for long email chains with different senders.For all those interested, this is an easy fix in 'app.js'. Simply modify the code as follows:// add the new signature string if (this.env.signatures && this.env.signatures[id]) { sig = this.env.signatures[id]; if (sig.indexOf('--')!=0) sig = '--\n'+sig; message += '\n'+sig; message = '\n\n'+sig+'\n\n'+message; }
editor.getBody().appendChild(sigElem);
editor.getBody().insertBefore(sigElem, editor.getBody().firstChild );