271.63 KB, image/png
181.84 KB, image/png
3.36 KB, patch
|Details | Diff | Splinter Review|
98.03 KB, image/png
179.08 KB, image/png
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.
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.
Summary: Content in Preferences > Advanced is cut off → Content in Preferences > Advanced is cut off in mac Retina display
Version: unspecified → 58 Branch
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
Attachment #8951849 - Flags: review?(jorgk)
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?
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 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+
I tried it but didn't get line breaks. Probably because the have to be in <hbox>.
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?
Attachment #8951849 - Flags: approval-comm-beta? → approval-comm-beta+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/comm-central/rev/4e30bc0739dd Remove unneeded checkForUpdatesButton in some cases in the Prefs Update pane. r=jorgk
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
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)
You need to log in before you can comment on or make changes to this bug.