PGP key manager keeps even multiple selection of items after deletion, which can end up in unintended deletion of more keys.
Categories
(MailNews Core :: Security: OpenPGP, defect, P2)
Tracking
(thunderbird_esr102+ fixed)
People
(Reporter: jschoett, Assigned: mkmelin)
Details
(Keywords: ux-error-prevention, ux-mode-error)
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
wsmwk
:
approval-comm-esr102+
|
Details | Review |
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:94.0) Gecko/20100101 Firefox/94.0
Steps to reproduce:
- Open PGP key manager window
- Select a key
- Delete the item using the 'Del' key
Actual results:
Item is deleted but another item is selected.
Especially if more items are selected, after deletion, other items are selected randomly.
Expected results:
The selection should disappear after deletion.
!! If I selected and deleted many items, it feels like no items were deleted (if the list keeps the selection) – and then I would press 'Del' again and delete the wrong items inadvertently!
Comment 1•4 years ago
•
|
||
Thanks jschoett for highlighting this problem.
Confirming based on comment 0, which is clear and plausible.
This is a datalossy UX error prevention fail, which can easily induce the user to delete more keys than intended, especially in the multiple-selection scenario. Also, if you press DEL key just a little too long, it will probably proceed to delete other keys which were not originally selected. Imo reporter is right that we must not select random other keys after deleting keys from the list. Instead, selection must be cleared after deletion.
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
It's pretty normal behavior that the next item is selected once something (like an email in a list) is deleted.
There is also a confirmation dialog you have to click through, so very little risk of accidental duplicate deletion.
For multi-select + delete I agree there's an odd behavior after confirmation.
Assignee | ||
Comment 3•3 years ago
|
||
Updated•3 years ago
|
Updated•3 years ago
|
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/b9de230b62e3
After deleting OpenPGP keys, clear the tree selection. r=kaie
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 5•3 years ago
|
||
Comment on attachment 9286352 [details]
Bug 1743697 - After deleting OpenPGP keys, clear the tree selection. r=kaie
[Approval Request Comment]
Regression caused by (bug #): not a regression
User impact if declined: can cause user error
Testing completed (on c-c, etc.): beta
Risk to taking this patch (and alternatives if risky): safe
Comment 6•3 years ago
|
||
Comment on attachment 9286352 [details]
Bug 1743697 - After deleting OpenPGP keys, clear the tree selection. r=kaie
[Triage Comment]
Approved for esr102
Comment 7•3 years ago
|
||
bugherder uplift |
Thunderbird 102.1.1:
https://hg.mozilla.org/releases/comm-esr102/rev/6ec84a2a79d9
Description
•