Usability loss in managing Cookies in Firefox 60

VERIFIED FIXED in Firefox 61

Status

()

defect
P1
normal
VERIFIED FIXED
Last year
Last year

People

(Reporter: bc, Assigned: mkohler)

Tracking

(Blocks 1 bug, {regression})

60 Branch
Firefox 61
Points:
---
Dependency tree / graph
Bug Flags:
qe-verify +

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox-esr60 wontfix, firefox59 unaffected, firefox60+ wontfix, firefox61+ verified)

Details

(Whiteboard: [storage-v2][triage])

Attachments

(1 attachment)

1. Open Preferences
2. Choose Privacy & Security
3. Select Manage Data under Cookies and Site Data
4. Select Cookie to be removed.
5. Click Remove Selected
6. The next site is not selected/highlighted.

In Firefox 59 and earlier the next site in the list would remain selected so that you could delete the next site by simply clicking Remove Selected again.

In Firefox 60, the selection is lost which requires you to click on the next site before clicking Remove Selected again.

This makes it a major PITA to review and delete unwanted cookies.

Once a site is selected and removed, the next site in the list should be selected so that the user can easily delete the next site in the list.

Allowing multiple selections would also be good, but would not replace the effectiveness of the previous behavior.
Yeah, I'm not sure what the best behavior would be here. FWIW, I just fixed multi-select with bug 1442183 and we could try to uplift that to 60.

We'll take a look at this.
See Also: → 1442183
Whiteboard: [storage-v2][triage]
Not sure whether this should really be a tracking issue for 60, but I filed for uplift in https://bugzilla.mozilla.org/show_bug.cgi?id=1442183#c9 which should help a ton with the usability, no matter what we decide to do in this bug (and when).
Happy to help out here as I've broken the UX here ;) Please n-i me once there is a decision on what to do here.
(In reply to Michael Kohler [:mkohler] from comment #3)
> Happy to help out here as I've broken the UX here ;) Please n-i me once
> there is a decision on what to do here.

I'll get ahead of triage meeting on Tuesday and say that we should probably just select the next website, since we explicitly warn the user of which websites are going to be deleted afterwards (so there shouldn't be much risk of accidentally deleting too much data).

Feel free to work on this, I'm happy to help if you need any :)

Thank you!
Assignee: nobody → me
fix-optional for 60 based on comment 2.
Select next item in list when removing items in Site Data Manager. When there are multiple selected sites, it will select the next item after the last previously selected item.
Thanks for the patch! That phabricator bridge isn't working particularly well... I'll review this later today
Flags: needinfo?(jhofmann)
Comment on attachment 8968673 [details]
Bug 1453589 - Select next item in list when removing items in Site Data Manager r=johannh

Johann Hofmann [:johannh] has approved the revision.

https://phabricator.services.mozilla.com/D965
Attachment #8968673 - Flags: review+
Flags: needinfo?(jhofmann)
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/525f1fb9501e
Select next item in list when removing items in Site Data Manager. r=johannh
Keywords: checkin-needed
I've fixed the error. Setting checkin-needed again. Sorry about that.
Flags: needinfo?(me)
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7c588027cbbb
Select next item in list when removing items in Site Data Manager. r=johannh
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/7c588027cbbb
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
This is much better. Thank you.
We built the Fx60 RC build today, so calling this too late for uplift consideration.
Verified as fixed with 61.0b9 on Ubuntu 14.4, MacOS 10.13 and Windows 10x64.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.