1438334 - Content in Preferences > Advanced is cut off in mac Retina display

Status: RESOLVED FIXED

(Thunderbird :: Preferences, defect)

Description

[Théo Chevalier]

Attachment: TB 59b1 French

It looks like bug 1155545 is back. Same configuration, but on macOS 10.13.3

I’ve been able to reproduce on 58b3, 59b1 and today’s Daily build.

Comment 1

[Eckard Berberich]

Attachment: Thunderbird 59 German locale preferences pane

Comment 2

[Eckard Berberich]

iMac Retina display running macOS 10.13.3.
Confirming this bug for the French and German locales of TB 58.0b3, TB 59.0b1 and Daily builds.

Comment 3

[Richard Marti (:Paenglab)]

Attachment: checkForUpdatesButton.patch

When I ported the update functionality from FX to TB, I haven't checked if all makes sense.

This bites us now because some states don't need the update button. And especially this states use longer text which make together with the not needed button the prefs pane very wide.

When the Admin has disabled updates or an other instance does updates or updates are no more supported it makes no sense to show the check update button.

Tested fix on German and French builds.

Comment 4

[Jorg K (CEST = GMT+2)]

I don't understand this bug very well. On Windows, the tabs of the advanced options are left-aligned and there is no cut-off.

I can't see any of the items adminDisabled, otherInstanceHandlingUpdates or unsupportedSystem, looks like there are hidden by code, but where? Will they ever be visible in TB?

I guess is doesn't hurt to remove disabled buttons of hidden items, but then, why not remove the entire item and any code referring to it.

Could you at least provide some screen shots?

Comment 5

[Richard Marti (:Paenglab)]

Attachment: widePrefs.png

The different items are in a deck. That means only one of them is shown and depending of the context the correct is shown. The deck uses the width of the widest item. The screenshot shows the widest item on French (and also German) TB. Removing the button "Rechercher des mises à jour" makes the whole deck fit again in the prefs window because this item uses then only the width of the text.

Comment 6

[Jorg K (CEST = GMT+2)]

Comment on attachment 8951849 [details] [diff] [review]
checkForUpdatesButton.patch

OK, I'm not 100% convinced that this is the best solution. It would be better to make sure the text is broken into multiple lines. I can imagine another language where this also won't fix even if you remove the buttons.

Comment 7

[Richard Marti (:Paenglab)]

I tried it but didn't get line breaks. Probably because the have to be in <hbox>.

Comment 8

[Richard Marti (:Paenglab)]

Comment on attachment 8951849 [details] [diff] [review]
checkForUpdatesButton.patch

It was found in beta and should also go to it to check if it's really fixed.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/4e30bc0739dd
Remove unneeded checkForUpdatesButton in some cases in the Prefs Update pane. r=jorgk

Comment 10

[Jorg K (CEST = GMT+2)]

TB 59 beta 2:
https://hg.mozilla.org/releases/comm-beta/rev/5a0ac769b0ddb7b35c85fd887fc81d270915bae1

Comment 11

[Eckard Berberich]

I just found a similar issue for the "Chat" and "Kalendar" (Calendar) prefs panes in TB 60rc/build4
(macOS 10.13.6 on a iMac 21.5" Retina display)

Comment 12

[Eckard Berberich]

Attachment: TB 60rc:build4.png showing cut off content in Calendar pref pane TB60rd/build4 German locale