Closed
Bug 1438334
Opened 7 years ago
Closed 7 years ago
Content in Preferences > Advanced is cut off in mac Retina display
Categories
(Thunderbird :: Preferences, defect)
Tracking
(thunderbird58 wontfix, thunderbird59 fixed, thunderbird60 fixed)
RESOLVED
FIXED
Thunderbird 60.0
People
(Reporter: tchevalier, Assigned: Paenglab)
References
Details
(Keywords: regression)
Attachments
(5 files)
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
patch
|
jorgk-bmo
:
review+
jorgk-bmo
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details |
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•7 years ago
|
||
Comment 2•7 years ago
|
||
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.
Updated•7 years ago
|
Summary: Content in Preferences > Advanced is cut off → Content in Preferences > Advanced is cut off in mac Retina display
Version: unspecified → 58 Branch
Updated•7 years ago
|
Flags: needinfo?(richard.marti)
Assignee | ||
Comment 3•7 years ago
|
||
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.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Flags: needinfo?(richard.marti)
Attachment #8951849 -
Flags: review?(jorgk)
Comment 4•7 years ago
|
||
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?
Assignee | ||
Comment 5•7 years ago
|
||
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•7 years ago
|
||
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.
Attachment #8951849 -
Flags: review?(jorgk) → review+
Assignee | ||
Comment 7•7 years ago
|
||
I tried it but didn't get line breaks. Probably because the have to be in <hbox>.
Keywords: checkin-needed
Assignee | ||
Comment 8•7 years ago
|
||
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.
Attachment #8951849 -
Flags: approval-comm-beta?
Updated•7 years ago
|
Attachment #8951849 -
Flags: approval-comm-beta? → approval-comm-beta+
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/4e30bc0739dd
Remove unneeded checkForUpdatesButton in some cases in the Prefs Update pane. r=jorgk
Updated•7 years ago
|
Target Milestone: --- → Thunderbird 60.0
Comment 10•7 years ago
|
||
Comment 11•6 years ago
|
||
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•6 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•