Closed Bug 1730740 Opened 3 years ago Closed 3 years ago

multimessage view is not fully scrollable

Categories

(Thunderbird :: Message Reader UI, defect)

Thunderbird 91
defect

Tracking

(thunderbird_esr91 wontfix)

RESOLVED FIXED
94 Branch
Tracking Status
thunderbird_esr91 --- wontfix

People

(Reporter: henry-x, Assigned: henry-x)

References

Details

(Keywords: regression, regressionwindow-wanted)

Attachments

(2 files)

Steps to reproduce

Click on a long thread of messages such that the multimessageview would need to be scrollable.

Result

Whilst a scrollbar does appear, scrolling to the bottom still leaves the end part cropped.

Note that this is the case in Classic, Wide and Vertical view.

Expect

The ability to scroll to and see the bottom of the multimessageview.

Blocks: tb91found
Version: unspecified → Thunderbird 91

I don't see any end part cropped.

I do see on trunk the multimessage view for vertical view is not scrollable.

Actally, not seeing that either (didn't have enough messages selected)

Attached image example of overflow (deleted) —

See the bottom right corner. The end of the scrollbar is cropped as well.

It's not cut of for me, but perhaps by chance. It looks like the scrollbar would go down beneath the status bar.

I think this is caused by the #content element having a height of 100% (which is the browser height), which doesn't account for the header height, and is adding 2em of padding on top of that.

This should use a column flex to distribute the available height. But there is something weird going on with the embedded html's styling, which is resisting a flex display. For example, the client height of the html and body is 0...

(In reply to Henry Wilkes [:henry] from comment #0)

Note that on Trunk, unlike 91, the Vertical view is completely unscrollable.

On checking, this isn't true, so I'll edit it out of the comment. It was just the email thread that I chose had enough vertical height to be cropped, but not enough to create the scrollbar. I.e. the height was greater than the available height, but less than the browser height (the 100% height given to the #content).

Seems some of the elements were positioned, which can also block a flex display. I think I can fix it.

Assignee: nobody → henry
Status: NEW → ASSIGNED
Target Milestone: --- → 94 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/43ae0c7ee327
Use flex column for the multimessage view - make it fully scrollable. r=mkmelin DONTBUILD

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: