Closed Bug 1861853 Opened 11 months ago Closed 11 months ago

Group headers have three lines in card view

Categories

(Thunderbird :: Folder and Message Lists, defect)

Thunderbird 120
defect

Tracking

(thunderbird_esr115 unaffected, thunderbird121 affected)

RESOLVED WONTFIX
121 Branch
Tracking Status
thunderbird_esr115 --- unaffected
thunderbird121 --- affected

People

(Reporter: rjwolke, Unassigned)

References

(Regression)

Details

(Keywords: regression, triaged)

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

Steps to reproduce:

Enable card view and group by sort in the message view.

Actual results:

Group headers have the same height as messages (which was recently increased to three lines).

Expected results:

Group headers should be reduced to a single line to reflect the fact that they only contain a signle line of text.

This does not affect touch navigation because the only clickable element in the group header is the arrow button.

Component: Message Reader UI → Folder and Message Lists

I can confirm that I'm also seeing this in v120.0b1

Thank you for reporting this. Because of how we currently calculate scrolling, cards are all the same height, even if they are headings and only one line of text.

I've spoken with Martin on the front-end team about this. From a technical standpoint, the card height is fixed based on font-size and density unless we significantly change the architecture of cards view.

I will also check in with the design team about any ways we can improve this visually.

We do plan to make some density and padding adjustments to the cards in bug 1858578. I will make sure Group by Sort is considered in the selected sizing.

Severity: -- → S4
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(elizabeth)
Keywords: triaged
See Also: → 1858578
Blocks: 1860631
Blocks: 1863403

We are not able to have less rows or a different height on different cards due to how the cards view is designed architecturally with the code. This is related to scrolling. See my earlier note in comment #2.

But for the sake of making this look better, that is something we can address. See bug 1858578 and bug 1863403.

Update: I also want to say thank you for reporting this. This is certainly not ideal in terms of how this looks. But we do want it to look better than it does now given the constraints.

No longer blocks: 1863403
Status: NEW → RESOLVED
Closed: 11 months ago
Flags: needinfo?(elizabeth)
Keywords: regression
Regressed by: 1855821
Resolution: --- → WONTFIX
See Also: → 1863403

Is bug 1861758 the same as this?

(In reply to Elizabeth Mitchell from comment #2)

the card height is fixed based on font-size and density unless we significantly change the architecture of cards view.

If there is a fault by design then the design needs to be corrected, it seems obvious.
It is not the end-users to endure such "mistake" by having GroupBy section over three line, instead of one which should be largely sufficient, just because something was wrongly coded. Yes it may demand a bit of effort from the dev team but it is likely to pay-off in the long term.

(In reply to Magnus Melin [:mkmelin] from comment #4)

Is bug 1861758 the same as this?

They are related, but different bugs.

Target Milestone: --- → 121 Branch
You need to log in before you can comment on or make changes to this bug.