Closed Bug 1539323 Opened 5 years ago Closed 5 years ago

Preferences tab should only be scrollable if overflowing

Categories

(Thunderbird :: Preferences, enhancement)

enhancement
Not set
normal

Tracking

(thunderbird68 fixed)

VERIFIED FIXED
Thunderbird 68.0
Tracking Status
thunderbird68 --- fixed

People

(Reporter: darktrojan, Assigned: darktrojan)

Details

Attachments

(1 file)

Currently all preference panes are the same height, because they're in a <deck>. This means we get a scrollbar if the tallest pane cannot fit in the space available. This looks silly for short panes.

Attachment #9053806 - Flags: review?(richard.marti)
Comment on attachment 9053806 [details] [diff] [review]
1539323-prefpane-deck-1.diff

Thanks.

Maybe in the future we could implement it the FX way with hiding the groupboxes of a pane. Then we can also add a search functionality.
Attachment #9053806 - Flags: review?(richard.marti) → review+
Keywords: checkin-needed
Target Milestone: --- → Thunderbird 68.0

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/e54ef7d0e92f
Stop displaying preferences in a <deck> to prevent unnecessary overflowing. r=Paenglab DONTBUILD

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED

Works for me in 68.0a1 April5 @Kubuntu 18.10

No scrolling in full size > I need a very small window to see a scroll bar.

Status: RESOLVED → VERIFIED
Resolution: FIXED → WORKSFORME

Please DO NOT change FIXED to WORKSFORME. The latter is for bugs that can't be reproduced, so some one says: This already works for me. When a fix was landed, the status MUST remain FIXED.

Status: VERIFIED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: WORKSFORME → FIXED

There is also no point changing the tracking flags. They are for tracking bugs where patches need backport/uplift. These are for the TB code manager to use and no one else.

Sorry, I wasn't sure how to give correct feedback. Thanks for information - won't do it again.

verified per hades

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

Attachment

General

Created:
Updated:
Size: