Closed Bug 1546282 Opened 5 years ago Closed 5 years ago

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

Categories

(Thunderbird :: General, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 68.0

People

(Reporter: khushil324, Assigned: khushil324)

References

Details

Attachments

(1 file, 1 obsolete file)

No description provided.
Assignee: nobody → khushil324
Type: defect → task
Attachment #9061221 - Flags: review?(mkmelin+mozilla)
Status: NEW → ASSIGNED
Comment on attachment 9061221 [details] [diff] [review]
Bug-1546282_remove-grid-am-smtp.patch

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

Looks ok to me, r=mkmelin

::: mailnews/base/prefs/content/am-smtp.xul
@@ +63,5 @@
>        <stack flex="1">
>          <spacer id="backgroundBox"/>
> +        <hbox>
> +          <vbox>
> +            <hbox pack="end" align="center" flex="1">

Also in this patch I think all the align="center" are not needed
Attachment #9061221 - Flags: review?(mkmelin+mozilla) → review+
Attachment #9061221 - Attachment is obsolete: true
Attachment #9061347 - Flags: review+
Keywords: checkin-needed

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/7a7af57de670
remove grid usage from am-smtp.xul. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 68.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: