1773783 - Various small UI styling issues in the message reader

Status: RESOLVED FIXED

(Thunderbird :: Message Reader UI, defect)

Description

[Alessandro Castellani [:aleca]]

There are a few small UI issue sin the message reader.
I'm gonna use this bug to collect them all and keep it open for potential future issues in 102.

• IMIP bar notification buttons are all attached together.
• The alignment of labels + text is not perfect when Asian characters are in the string.

Comment 1

[Alessandro Castellani [:aleca]]

Attachment: align-message.eml

Test message with mixed characters to use for testing.

Comment 2

[Alessandro Castellani [:aleca]]

Attachment: Bug 1773783 - Fix recipients line height for mixed characters and imip bar buttons spacing. r=Paenglab

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/c6d6c6a78491
Fix recipients line height for mixed characters and imip bar buttons spacing. r=Paenglab

Comment 4

[Alessandro Castellani [:aleca]]

Comment on attachment 9280766 [details]
Bug 1773783 - Fix recipients line height for mixed characters and imip bar buttons spacing. r=Paenglab

[Approval Request Comment]
Regression caused by (bug #): bug 1556261
User impact if declined: Wrong text alignment in header labels and missing spacing in IMIP bar buttons
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): low

Comment 5

[Wayne Mery (:wsmwk)]

Comment on attachment 9280766 [details]
Bug 1773783 - Fix recipients line height for mixed characters and imip bar buttons spacing. r=Paenglab

[Triage Comment]
Approved for beta

Comment 6

[Rob Lemley [:rjl]]

Thunderbird 102.0beta6:
https://hg.mozilla.org/releases/comm-beta/rev/ea1a48734353

Comment 7

[Geoff Lankow (:darktrojan)]

Attachment: Bug 1773783 - Fix avatar placeholders where multi-byte characters are used. r=aleca

Just in case people have names containing emoji. 🤪

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/a95e0235125b
Fix avatar placeholders where multi-byte characters are used. r=aleca

Comment 9

[Geoff Lankow (:darktrojan)]

Comment on attachment 9281993 [details]
Bug 1773783 - Fix avatar placeholders where multi-byte characters are used. r=aleca

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined:
Testing completed (on c-c, etc.):
Risk to taking this patch (and alternatives if risky):

Comment 10

[Wayne Mery (:wsmwk)]

Comment on attachment 9281993 [details]
Bug 1773783 - Fix avatar placeholders where multi-byte characters are used. r=aleca

[Triage Comment]
Approved for beta

Comment 11

[Alessandro Castellani [:aleca]]

Let's close this to avoid spanning this bug throughout multiple versions.