1817030 - consider changing the overscroll effect

Status: RESOLVED FIXED

(Thunderbird :: Folder and Message Lists, defect)

Description

[Sören Hentzschel]

Attachment: overscroll effect

I am not a big fan of overscroll effects, especially in native applications. With Supernova there is now an overscroll effect in the folder pane as well as in the message list, both are not present in Thunderbird 102. Since it's just my personal preference I don't want to argue if it should be disabled or not. It would be easy to disable via CSS if you wanted to do that (overscroll-behavior: none).

But I don't think that it makes sense that the header row of the message list is moving with the mail rows. I think that the header row should be fixed and that the overscroll effect should only apply to the mail rows.

I attached a screenshot / "mockup" to demonstrate what I mean.

Comment 1

[Alessandro Castellani [:aleca]]

This should have been fixed since we updated the row height and count calculation, so that empty row shouldn't happen.
Can you still reproduce this?

Comment 2

[Sören Hentzschel]

No, this is not fixed. This issue is not about an empty row but the so called overflow effect. As long as the effect is enabled there will always be an empty space during the scrolling, that's the expected behaviour.

If you're already on the top of the message list and try to scroll further to top you can see this effect. Actually it includes the header row so the empty space appears above the header row. But I think that it makes much more sense to have a fixed header row so that the empty space appears below the header row. Alternatively the effect can be disabled with overscroll-behavior: none if you prefer that.

Comment 3

[Andrei Hajdukewycz [:sancus]]

This is Mac only, doesn't happen on Windows.

Comment 4

[Alessandro Castellani [:aleca]]

Attachment: Bug 1817030 - [macOS] Remove the overscroll effect on the tree view. r=#thunderbird-front-end-reviewers

Comment 5

[Alessandro Castellani [:aleca]]

Let's remove the overscroll effect altogether from the tree list since we don't use it for refresh or anything else, and the table header needs to stay sticky.

Comment 6

[Gregory Pappas [:gregp]]

(In reply to Andrei Hajdukewycz [:sancus] from comment #3)

This is Mac only, doesn't happen on Windows.

Overscroll was enabled upstream on Windows last month in bug 1810641

Comment 7

[Alessandro Castellani [:aleca]]

Thanks for the info Gregory, I disabled that for all platforms.
Try run if anyone wants to try it (after it finished building) https://treeherder.mozilla.org/jobs?repo=try-comm-central&revision=bc223619b787306b80be2b427e8b09709df34e80

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/067a5adf83d7
[macOS] Remove the overscroll effect on the tree view. r=sancus

Comment 9

[Sören Hentzschel]

Let's remove the overscroll effect altogether from the tree list since we don't use it for refresh or anything else

Then you might consider to remove the overscroll effect from the folder pane as well.

Comment 10

[Wayne Mery (:wsmwk)]

(In reply to Sören Hentzschel from comment #9)

Let's remove the overscroll effect altogether from the tree list since we don't use it for refresh or anything else

Then you might consider to remove the overscroll effect from the folder pane as well.

For consistency might be a good idea. But afaict, so far, the overscroll in folder pane does not cause any problems?

Comment 11

[Sören Hentzschel]

No, it does not cause any problems, it just doesn't serve any real purpose. :)

Comment 12

[Alessandro Castellani [:aleca]]

Continuing the work in bug 1830557