Closed martgil closed 1 month ago
Hello @sosnovsky - may I hear your feedback regarding this whenever your are available. Once confirmed that this looks good to you, I'll be sharing a custom build to the reporting users for them to see it working on their side before we can merge this. Thanks!
Hi @martgil - I tried to reproduce mentioned bug on master
build, but everything worked well for me.
Maybe you can share video of this issue by email?
You can also share this build with reporting users, to check if provided fix works for them.
Yes, Roma - I just sent a screencast over secure email. Thanks!
I will also create a custom build now and share to the reporting users.
This PR adds check ensuring that rich-text is enable before ever removing rich text formatting. This solves the issue of having a prepended newlines when typing, for example, "HelloEnterWorld" where it produces
in the
contenteditable.div
in the master branch.To test this manually:
1.) Open Gmail with FlowCrypt browser extension installed and configured. 2.) Send you're self a signed message email. 3.) Reply to your own signed message email. (The bug appears at this point on "master" branch but not here.) 4.) Type "HelloEnterWorld" and it will produce output on
contenteditable.div
correctly with:instead of the bugged version:
close #5599
Tests (delete all except exactly one):
master
and this branch. So looks like the bug is particularly be reproducible through actual usage. I can share this build to the reporting users for them to double check it -- though it works perfectly when i tested it.To be filled by reviewers
I have reviewed that this PR... (tick whichever items you personally focused on during this review):