Closed Bug 1776597 Opened 2 years ago Closed 2 years ago

Text in newsgroup download dialogue misaligned

Categories

(Thunderbird :: Theme, defect)

Thunderbird 102
defect

Tracking

(thunderbird_esr102 fixed, thunderbird102 wontfix, thunderbird103 fixed)

RESOLVED FIXED
104 Branch
Tracking Status
thunderbird_esr102 --- fixed
thunderbird102 --- wontfix
thunderbird103 --- fixed

People

(Reporter: rachel, Assigned: Paenglab)

References

(Blocks 1 open bug)

Details

(Whiteboard: [TM:102.0.1optimistic])

Attachments

(5 files)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Firefox/91.0

Steps to reproduce:

See attachment.

Blocks: tb102found
Attached image download-Headers.png (deleted) —

You mean the newsgroup label? Here it doesn't look like yours.

Some padding is missing around the content.

This was done with the TB 102 RC.

MIne from 102 RC too.

Strange. No change with userChrome.css disabled, not that it had any padding changed.

margin-inline-start is 0, so why is it a surprise that it's not indented? If you look closely, the m from mozilla in your screenshot isn't correct either.

Hmm, I have no rule from tabs.css. Do you have an extension installed?

Status: UNCONFIRMED → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID

We forgot to say that LCT were injecting the entire style sheet as author sheet. But even after fixing that we now get margin-inline-start of 4px which still looks bad. It should be 6px like the description text below.

Status: RESOLVED → UNCONFIRMED
Resolution: INVALID → ---
Assignee: nobody → richard.marti
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Target Milestone: --- → 104 Branch

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/297ea54fb7cd
Give the news download dialog a padding and align the labels. r=aleca

Status: ASSIGNED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED

Comment on attachment 9283192 [details]
Bug 1776597 - Give the news download dialog a padding and align the labels. r=aleca

[Approval Request Comment]
User impact if declined: not enough padding on the dialog
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): low

Attachment #9283192 - Flags: approval-comm-esr102?
Attachment #9283192 - Flags: approval-comm-beta?

Comment on attachment 9283192 [details]
Bug 1776597 - Give the news download dialog a padding and align the labels. r=aleca

[Triage Comment]
Approved for beta

Attachment #9283192 - Flags: approval-comm-beta? → approval-comm-beta+

optimistically approving for TB102.0.1 - assuming it will pass beta over the next several days

Whiteboard: [TM:102.0.1optimistic]

Comment on attachment 9283192 [details]
Bug 1776597 - Give the news download dialog a padding and align the labels. r=aleca

[Triage Comment]

Attachment #9283192 - Flags: approval-comm-esr102? → approval-comm-esr102+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: