ProtonMail / WebClients

Monorepo hosting the proton web clients
GNU General Public License v3.0
4.53k stars 571 forks source link

<Esc> in outgoing-link prompt should not close email #227

Closed tomsmeding closed 3 years ago

tomsmeding commented 3 years ago

Not sure if bug or feature request, hence reporting as the latter.

Is your feature request related to a problem? Please describe. Pressing the escape key while the pop-up dialog for an external link in an email is visible, currently closes not only the pop-up dialog but also the email. This happens both in the full-page (row) email layout and the vertical-split (column) layout. This compounds with the related issue that pressing enter in the pop-up dialog opens the external link, but does not close the dialog. I regularly get emails (think notification emails) where I want to open a link in the email, do stuff on the webpage, close the page, and then delete the email. Avoiding the mouse, I open the email (in full-page layout), click the link somehow, and press enter. When I come back to the protonmail tab later, I press escape to close the dialog which is still open because of said related issue, and press t to trash the email — which works, I think, but closes the email first and then trashes it in the inbox view. This always scares me because I have no proof whatsoever that I deleted the right email, making me undo and try again explicitly.

[related: thanks heaps for 'undo'.]

I'm probably reflexively doing some actions no-one else does regularly, but I do think some usability improvements are possible here.

Describe the solution you'd like

Alternatively, or additionally:

Describe alternatives you've considered Trashing with t not working in the inbox view without first explicitly selecting the email (e.g. using x). This would make the interface feel safer to me (the highlight border on the left is fairly arbitrarily placed sometimes, and also a minor UI element, while selecting an email using the checkbox or x feels like the right amount of ceremony to justify deleting an email using one keypress). However, this would be a breaking change that has a far higher probability of breaking people's workflows, I think, than the alternatives I described above.

Additional context (none)