Closed Bug 1509209 Opened 11 months ago Closed 11 months ago

Superfluous scrollbars in the trackers list subpanel

Categories

(Firefox :: Site Identity, defect, P3)

defect

Tracking

()

VERIFIED FIXED
Firefox 65
Tracking Status
firefox65 --- verified

People

(Reporter: bruce.bugz, Assigned: bruce.bugz)

References

(Blocks 1 open bug)

Details

(Whiteboard: [privacy65])

Attachments

(2 files)

Attached image Capture.PNG
Screenshot taken on http://forums.mozillazine.org/viewtopic.php?f=23&t=3043920
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P3
Whiteboard: [privacy65]
Blocks: 1501991
Blocks: 1509215
Changing `overflow` from `scroll` to `auto` on `#identity-popup-trackersView-list` seems to show these scrollbars only when needed.

If that's the correct fix, I can submit a patch.
Yes, you're right. Happy to take a patch. Thanks!
Assignee: nobody → bruce.bugz
Status: NEW → ASSIGNED
Attached patch bug1509209.patchSplinter Review
Bug 1509209 - Show scrollbars in the Trackers sub-panel only if the content overflows. r=johannh
Attachment #9027815 - Flags: review?(jhofmann)
Comment on attachment 9027815 [details] [diff] [review]
bug1509209.patch

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

Thanks!
Attachment #9027815 - Flags: review?(jhofmann) → review+
Pushed by rmaries@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/34b22b4e2b02
Show scrollbars in the Trackers sub-panel only if the content overflows. r=johannh
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/34b22b4e2b02
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
Verified on Windows 10, Ubuntu 18, Mac OS, on Nightly 65.0a1 (2018-11-27).
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.