554404 - SMTP editor dialog misaligned after Bug 525238 landed

Status: RESOLVED FIXED

(Thunderbird :: Account Manager, defect)

Description

[Alex Ihrig [:Thunderbird_Mail_DE]]

Attachment: Patch to remove hbox element

In SMTP editor dialog the last line (Username: _______________) is misaligned because of an <hbox class="indent"> element.

IMHO the <hbox> element could be removed. I'll attach a patch.

Comment 1

[Alex Ihrig [:Thunderbird_Mail_DE]]

Attachment: Patch to remove hbox element (encoding fix)

Comment 2

[Ben Bucksch (:BenB)]

Please show a screenshot documenting what you mean with "misaligned", and how your patch changes it.

IIRC, I intentionally intended the username, because it is dependent to the authentication, and it also was intended in the old dialog for the same reason.

Comment 3

[Ben Bucksch (:BenB)]

Attachment: Before auth patch

This is how it looked before my change

Comment 4

[Ben Bucksch (:BenB)]

Attachment: After auth patch, without this bug's patch

This is how it looks after bug 525238 and without your patch.

Granted, it's not pretty, but it's intentional, not a bug, and you can see it preserves the idea of the old dialog.

I don't have an opinion of whether it's better with or without the indention. I agree it would look better without.

Comment 5

[Alex Ihrig [:Thunderbird_Mail_DE]]

(In reply to comment #2)
> IIRC, I intentionally intended the username, because it is dependent to the
> authentication, and it also was intended in the old dialog for the same reason.

Okay, you're right.

(In reply to comment #4)
> I don't have an opinion of whether it's better with or without the indention.
> I agree it would look better without.

Maybe only the "centered" looking string "User Name:" confused me.

From my point of view we can close this bug.

Comment 6

[Ben Bucksch (:BenB)]

I'm undecided. I leave it to Bryan.

Comment 7

[WADA:World Anti-bad-Duping Agency]

Can you add message submission port number in new panel upon panel design/layout change?
> Port: [ 25 ] Default: 25
>              Message Submission Port: 587

Question about "StartTLS, if avail" in sample screen shot.
Will "StartTLS, if avail" be enabled again to resolve StartTLS variant of bug 534158?
  No STLS to EHLO, if connection from internal network.
  STLS to EHLO and SMTP forces STLS, if connection from external network.
(bug 534158)
  No AUTH to EHLO, if connection from internal network.
  AUTH to EHLO and SMTP forces SMTP-AUTH, if connection from external network.

Question relates to solution of bug 534158.
Is there any room for option like "Use name and password, if requested" in addition to current "Use name and password"(always try SMTP-AUTH))?

Comment 8

[Ben Bucksch (:BenB)]

WADA, all of this is offtopic here. Please keep one bug to one issue. And questions should go per mail, not bugzilla. I reply per email.

Comment 9

[Nomis101]

Attachment: Screenshot with and without the patch

I've today encountered the same thing and I agree with Alex, its really misaligned.
Here is a screenshot from a build without and with the patch from #1, so you can see the differences.

Comment 10

[Bryan Clark (DevTools PM) [:clarkbw]]

Comment on attachment 434291 [details] [diff] [review]
Patch to remove hbox element (encoding fix)

didn't try this out but it looks good from the screen shot and the code change makes sense.

Comment 11

[Ben Bucksch (:BenB)]

Comment on attachment 434291 [details] [diff] [review]
Patch to remove hbox element (encoding fix)

ok, then r=BenB

Comment 12

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 434291 [details] [diff] [review]
Patch to remove hbox element (encoding fix)

It looks fine to me, but I'm not actually qualified to review this bit of the code.  ;)

Perhaps bienvenu would give it a look?

Comment 13

[[e-mail]]

Comment on attachment 434291 [details] [diff] [review]
Patch to remove hbox element (encoding fix)

[I didn't notice on the original bug, but you probably didn't need a separate hbox just for the indent; adding class="indent" to the label should have worked.]

Comment 14

[Ben Bucksch (:BenB)]

Commited as <http://hg.mozilla.org/comm-central/rev/8d802c091f7c>

FIXED