Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Secure reply issues when composing signed message #5599

Closed
martgil opened this issue Feb 12, 2024 · 3 comments · Fixed by #5729
Closed

Secure reply issues when composing signed message #5599

martgil opened this issue Feb 12, 2024 · 3 comments · Fixed by #5729
Assignees
Milestone

Comments

@martgil
Copy link
Collaborator

martgil commented Feb 12, 2024

Description

When replying to a signed message, pressing enter creates a new line above the current line the user is writing wherein it should create a new line below the current line. Aside from this, pressing backspace does nothing making any typos undeletable.

This only happens on messages composed in a signed message and composing encrypted + signed is not affected.

Reference:
https://mail.google.com/mail/u/human@flowcrypt.com/#inbox/FMfcgzGxRdxBwtHmFKSTKPNLSQSsnqKm
https://mail.google.com/mail/u/human@flowcrypt.com/#inbox/FMfcgzGxRfBSrqRfqcSwksGTzqkVdMhQ
https://mail.google.com/mail/u/human@flowcrypt.com/#inbox/FMfcgzGxSRMQMmRDKjttmnsQMWRQGDGb

@martgil martgil added this to the 8.5.4 milestone Feb 12, 2024
@martgil
Copy link
Collaborator Author

martgil commented Feb 13, 2024

I noticed just now that when trying to send a blank signed message, there was this unhandled error message that occurred.

image

Edit: I tried to replicate this particular sub-issue once more and it wasn't occurring anymore.

@sosnovsky
Copy link
Collaborator

I noticed just now that when trying to send a blank signed message, there was this unhandled error message that occurred.

Oh, please fix it when working on the initial issue

@martgil
Copy link
Collaborator Author

martgil commented Feb 13, 2024

Yes, i will cover it as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants