Closed Bug 1462418 Opened 6 years ago Closed 6 years ago

Delete key functionality in the Cookies and Site Data Manager

Categories

(Firefox :: Settings UI, defect, P3)

60 Branch
defect

Tracking

()

RESOLVED FIXED
Firefox 64
Tracking Status
firefox60 --- wontfix
firefox61 --- wontfix
firefox62 --- wontfix
firefox63 --- wontfix
firefox64 --- fixed

People

(Reporter: remtanmajitenshi, Assigned: evilpie)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Same as Bug 1406679, but about Manage Data dialog in about:preferences#privacy. Now that old cookie manager is gone, it's even more important. Current UX is awful, all users are forced to use mouse: click on item, then move cursor to Remove Selected button and click, then selecting is gone (not switched to next string) and user needs to select another item again by mouse, instead of moving up and down by arrow keys.
Behavior should be the same as in the old cookie manager: it should be possible to work with keyboard only, delete key on selected item should behave like Remove Selected button, after this selecting should switch to next item (if present, or previous item if deleted one was last in the list). Then user would be able to:
— move very fast by keyboard arrows and remove items that he wants to remove via delete key (keyboard only)
— or use mouse only for selecting items and remove them via delete key (keyboard and mouse on different tasks)
— or use mouse only on Remove Selected button and do selecting via arrow keys (keyboard and mouse on different tasks)
Currently all those three approaches are impossible.
Firefox 62, unlike 60 version, does switch selecting to next item after deleting, but loses focus or something like that, so ability to move up and down by arrow keys is still disabled right after deleting.
And one more problem. If I do select and remove some items, and accidentally misclick to area outside of the dialog before saving changes, then I am losing all my progress without any warnings.
We should probably focus this bug on the delete key thing. I agree that would be nice to have. Feel free to file more bugs on the other issues :)
Blocks: storage-v2
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P3
Assignee: nobody → evilpies
Status: NEW → ASSIGNED
Attached patch Test. r?Splinter Review
Attachment #9006190 - Flags: review?(jhofmann)
Attachment #9006192 - Flags: review?(jhofmann)
Comment on attachment 9006190 [details] [diff] [review]
Implement Delete key functionality to remove site data. r?

Review of attachment 9006190 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with the correct key handling on OSX (I suppose you don't need another review for that)

Thanks!

::: browser/components/preferences/siteDataSettings.js
@@ +290,4 @@
>    onKeyPress(e) {
>      if (e.keyCode == KeyEvent.DOM_VK_ESCAPE) {
>        this.close();
> +    } else if (e.keyCode == KeyEvent.DOM_VK_DELETE) {

Can you please make this handle the backspace key on OSX?

https://searchfox.org/mozilla-central/rev/5a18fb5aeeec99f1ca1c36a697082c221189a3b9/browser/components/preferences/sitePermissions.js#257-259
Attachment #9006190 - Flags: review?(jhofmann) → review+
Attachment #9006192 - Flags: review?(jhofmann) → review+
https://hg.mozilla.org/mozilla-central/rev/b736431b1255
https://hg.mozilla.org/mozilla-central/rev/4b51290fb5df
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: