Skip to content

[WIP] Fix the 'Reply' button behavior, align it with Gmail#4101

Closed
limonte wants to merge 7 commits intomasterfrom
issue-4077-reply-button
Closed

[WIP] Fix the 'Reply' button behavior, align it with Gmail#4101
limonte wants to merge 7 commits intomasterfrom
issue-4077-reply-button

Conversation

@limonte
Copy link

@limonte limonte commented Nov 8, 2021

This PR aligns the behavior of the 'Reply' button with Gmail:

  • if from !== acctEmail => reply to the from email address only
  • if from === acctEmail then reply to all to email addresses (excluding cc and bcc)

close #4077

TODO: tests


Tests (delete all except exactly one):

  • Tests added or updated

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

@limonte limonte force-pushed the issue-4077-reply-button branch from 2478c5a to ddd2d3c Compare November 9, 2021 19:20
@limonte limonte force-pushed the issue-4077-reply-button branch from a39b3b2 to 77d78fb Compare November 9, 2021 21:28
@limonte
Copy link
Author

limonte commented Nov 9, 2021

Closing in favor of #4107

@limonte limonte closed this Nov 9, 2021
@limonte limonte deleted the issue-4077-reply-button branch November 11, 2021 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The "reply" button should only reply to the sender, it currently works the same way as "reply all" button

1 participant