#238 new

Signature inserted every time message body is reached via Tab key in composer

Reported by larrycz | February 24th, 2012 @ 07:42 AM

Version: 2759

Procedure: Open composer (Cmd+N). Press Tab until you reach the body of the message. Delete the signature from the text. Press Shift+Tab until you reach the To address field. Add some known address. Press Tab again until you reach the message body.

Expected behaviour: The signature will not be re-added.

Actual behaviour: The signature will reappear each time you navigate to the message body, over and over again.

Note: It doesn't make sense for the composer to add the signature on every entering of the message body. Only do it the first time.

Comments and changes to this ticket

  • benny

    benny February 24th, 2012 @ 11:05 AM

    • State changed from “new” to “accepted”

    It is not the entering of the message body which triggers the addition. It is the change of the To header and that is expected behavior.

    The problem is that MailMate does not detect that you explicitly deleted the signature. If you use the Signature menu (⌃⌘S) to select “No Signature” then MailMate respects your choice.

    I'll see if I can improve this such that if the body has been manually edited to remove a signature then a new signature is not inserted even if changing the recipient.

  • larrycz

    larrycz February 24th, 2012 @ 05:04 PM

    Actually, if I type the address in, enter the message body, then go back (still using Tab) and re-enter the message body without changing the address field, the signature still reappears. In that particular case it is the entering of the message body that triggers the behaviour.

  • benny

    benny March 1st, 2012 @ 10:16 AM

    • State changed from “accepted” to “fixcommitted”

    I still think the problem is related to tabbing out of the To header. I've now changed the code such that if the To header has not been changed then the signature is not updated (added in this case).

    You are welcome to reopen the ticket if it does not behave as expected in the next update.

  • benny

    benny March 21st, 2012 @ 04:40 PM

    • State changed from “fixcommitted” to “fixreleased”

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Mac OS X email client.

Shared Ticket Bins

People watching this ticket