1546279 - remove grid usage from comm/mailnews/base/prefs/content/am-main.xul

Status: RESOLVED FIXED

(Thunderbird :: General, task)

Comment 1

[Khushil Mistry [:khushil324]]

Attachment: Bug-1546279_remove-grid_am-main.patch

Comment 2

[Khushil Mistry [:khushil324]]

Attachment: Screenshot 2019-04-29 at 4.16.32 PM.png

I guess what you are talking about in https://bugzilla.mozilla.org/show_bug.cgi?id=1545907#c2 is this dialogue box. I have corrected the behavior over here. You can check it.

Comment 3

[Khushil Mistry [:khushil324]]

Attachment: Bug-1546279_remove-grid_am-main.patch

Comment 4

[Magnus Melin [:mkmelin]]

Comment on attachment 9061327 [details] [diff] [review]
Bug-1546279_remove-grid_am-main.patch

Review of attachment 9061327 [details] [diff] [review]:
-----------------------------------------------------------------

r=mkmelin

::: mailnews/base/prefs/content/am-main.xul
@@ +41,5 @@
>        <description>&identityDesc.label;</description>
>        <separator class="thin"/>
> +      <hbox>
> +        <vbox>
> +          <hbox flex="1" align="center">

for these hboxes you can remove the align="center"

Comment 5

[Khushil Mistry [:khushil324]]

It does not look good without align="center". So we need this in this patch.

Comment 6

[Khushil Mistry [:khushil324]]

Attachment: Screenshot 2019-04-29 at 7.38.54 PM.png

It will look something like this without align=center.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/9be27d28fd45
remove grid usage from am-main.xul. r=mkmelin