Closed Bug 1802706 Opened 4 months ago Closed 4 months ago

Confirmation window to delete cookies and website data is shown incorrectly


(Firefox :: Settings UI, defect)

Firefox 109



109 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox107 --- unaffected
firefox108 --- verified
firefox109 --- verified


(Reporter: kt+bugzilla, Assigned: emilio)




(Keywords: regression)


(3 files)

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:109.0) Gecko/20100101 Firefox/109.0

Steps to reproduce:

I have a german Nightly version, the English translation may be different.
Go to preferences, Data and Security (the menu with the lock symbol), Cookies and Website data area, and click manage data.
Issue 1 (clipboard2 image):
Delete the data of at least 50+ sites and click save changes.
The Button to delete them permanently is outside the visible screen area as it should be a list where I can scroll down, the window size is for the full length of the list.
Issue 2 (clipboard3 image):
delete for example the data from three sites and without really checking you see only 2, even as there is more than enough free space in the window to show all.

Actual results:

Issue 1:
It should show a list that can be scrolled down instead of a window that is too long.
Issue 2:
It should show all websites and not hide a site.

Expected results:

see actual results

Attached image Clipboard02.jpg
Attached image Clipboard03.jpg

The Bugbug bot thinks this bug should belong to the 'Core::Networking: Cookies' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: Untriaged → Networking: Cookies
Product: Firefox → Core
Component: Networking: Cookies → Settings UI
Product: Core → Firefox
21:38.13 INFO: Narrowed integration regression window from [bf1bc477, a913c869] (3 builds) to [bf1bc477, 1e4afe94] (2 builds) (~1 steps left)
21:38.13 INFO: No more integration revisions, bisection finished.
21:38.13 INFO: Last good revision: bf1bc477f95d9ca0686bf22e577727d11be616fc
21:38.13 INFO: First bad revision: 1e4afe94c89bf0d123accce7f6fe4565be09d5c0
21:38.13 INFO: Pushlog:

Hey emilio, looks like there's some fallout from bug 1796915 related to the cookie removal confirmation dialog. Got a second to look at it?

Flags: needinfo?(emilio)
Regressed by: 1796915
Ever confirmed: true

Of course!

Assignee: nobody → emilio
Flags: needinfo?(emilio)

TIL we had preferences dialogs that weren't subdialogs!

contain: size obviously doesn't work on those, and since they're not
subdialogs they don't have the "make them scroll if too tall" code,
causing this.

Converting this one is trivial tho, so do that.

Pushed by
Convert site data settings confirmation dialog to modern flex layout. r=mconley,settings-reviewers
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch

The patch landed in nightly and beta is affected.
:emilio, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox108 to wontfix.

For more information, please visit auto_nag documentation.

Flags: needinfo?(emilio)

Comment on attachment 9305648 [details]
Bug 1802706 - Convert site data settings confirmation dialog to modern flex layout. r=mconley,Gijs,dao

Beta/Release Uplift Approval Request

  • User impact if declined: comment 0
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: comment 0
  • List of other uplifts needed: none
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Very-scoped CSS fix.
  • String changes made/needed: none
  • Is Android affected?: No
Flags: needinfo?(emilio)
Attachment #9305648 - Flags: approval-mozilla-beta?
Flags: qe-verify+

Comment on attachment 9305648 [details]
Bug 1802706 - Convert site data settings confirmation dialog to modern flex layout. r=mconley,Gijs,dao

Approved for 108.0b9

Attachment #9305648 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
QA Whiteboard: [qa-triaged]

I have reproduced this issue on Win 7 x64, using STR from comment 0, on an affected Nightly build (2022-11-26).

The issue is verified as fixed on the latest builds, Beta 108 and Nightly 109, under Win 10 x64 and Win 7 x64.

QA Whiteboard: [qa-triaged]
Flags: qe-verify+
Regressions: 1814063
You need to log in before you can comment on or make changes to this bug.