1586572 - Opening Account Settings, server settings for an IMAP account, the selected trash folder field is too small

Status: RESOLVED FIXED

(Thunderbird :: Account Manager, defect)

Description

[Alice0775 White]

Attachment: screenshot, 67.0b1 vs 68.1.1

+++ This bug was initially created as a clone of Bug #1586567 +++

The widget is too small on TB68.1.1, see screenshot

Working OK on TB 67 beta 1.

Comment 1

[Alice0775 White]

Regression window:
https://hg.mozilla.org/comm-central/pushloghtml?fromchange=4054e820674ca76020fee6b2b747700bd2cbed03&tochange=17a0eac83bc55c87a6f0dbe706c78b06d99a8354
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=083106d8fc7407c880a3a044c83d4e15e5961063&tochange=083106d8fc7407c880a3a044c83d4e15e5961063

Regressed by: 17a0eac83bc55c87a6f0dbe706c78b06d99a8354 Khushil Mistry — Bug 1546281 - remove grid usage from am-server.xul. r=mkmelin DONTBUILD

Comment 2

[Richard Marti (:Paenglab)]

Attachment: 1586572-am-server-menulist.patch

This fixes it for me. And it seems to fix bug 1586567 too, at least for me.

Comment 3

[Jorg K (CEST = GMT+2)]

Comment on attachment 9099151 [details] [diff] [review]
1586572-am-server-menulist.patch

Thanks, that works. Do we really need this for TB 68? On my TB 68.1.2 the box doesn't look too small.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/bb1bd474fda9
Let the msgTrashFolderPicker menulist flex to show more of the label. r=jorgk DONTBUILD

Comment 5

[Richard Marti (:Paenglab)]

(In reply to Jorg K (GMT+2) from comment #3)

Comment on attachment 9099151 [details] [diff] [review]
1586572-am-server-menulist.patch

Thanks, that works. Do we really need this for TB 68? On my TB 68.1.2 the
box doesn't look too small.

On my TB 68 the menulist shows Ordner...ählen... and with the patch Ordner wählen.... And when I make the string longer all is shown, until the max-width of 300px.

Comment 6

[Jorg K (CEST = GMT+2)]

Comment on attachment 9099151 [details] [diff] [review]
1586572-am-server-menulist.patch

OK, thanks for testing.

Comment 7

[Jorg K (CEST = GMT+2)]

TB 68.1.2 ESR or TB 68.2:
https://hg.mozilla.org/releases/comm-esr68/rev/f1f9af28441770cec04af0abd6dd372e6d45610e

Comment 8

[Jorg K (CEST = GMT+2)]

TB 70 beta 4:
https://hg.mozilla.org/releases/comm-beta/rev/d0b0f5b31baefecfcacd501a24723a0ab451c83a