Closed Bug 1710407 Opened 3 years ago Closed 3 years ago

Autofill address/credit card dialogs are unnecessarily scrollable

Categories

(Firefox :: Settings UI, defect, P3)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
90 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox88 --- unaffected
firefox89 --- unaffected
firefox90 --- verified

People

(Reporter: Gijs, Assigned: Gijs)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression, Whiteboard: [proton-modals])

Attachments

(1 file)

The 6px padding on :root added in bug 1705330 is causing these dialogs to scroll by 12px vertically.

I don't really understand why, off-hand - perhaps the scrollHeight for the document isn't taking the padding into account, and we should be adding the padding to body instead, for the HTML dialog case? I also noticed that bug 1684530 solved the same problem as bug 1705330 but only for the autofill dialogs.

These kinds of bugs would stand a better chance of not being introduced if these features were default-enabled...

Whiteboard: [proton-modals]
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Priority: -- → P3

I messed up the initial tracking flags here, this doesn't affect 89 anyways, because the regressing bug, bug 1705330, wasn't uplifted.

Severity: -- → S4
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/217eac223b79 fix unnecessary scrollbars in address/creditcard dialogs in the settings, r=jaws,preferences-reviewers
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
QA Whiteboard: [qa-90b-p2]

I have reproduced this on an affected Nightly build 90.0a1 (2021-05-10)

Confirming this issue as verified fixed on Win 10x64, Ubuntu 20.04 and macOS 10.15, using Firefox 90.0b5 (2021-06-08).
Thanks!

Status: RESOLVED → VERIFIED
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: