Bug 393157 - Poor smooth scrolling performance
Summary: Poor smooth scrolling performance
Status: RESOLVED WORKSFORME
Alias: None
Product: Elisa
Classification: Applications
Component: general (show other bugs)
Version: 0.1
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-04-15 08:29 UTC by Kushnir Bogdan
Modified: 2018-09-05 19:26 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kushnir Bogdan 2018-04-15 08:29:27 UTC
How to disable these functions?
Comment 1 Nate Graham 2018-04-15 22:15:15 UTC
This is a bug tracker, not a user support hotline. See https://community.kde.org/Get_Involved/Bug_Reporting#Step_1:_Make_sure_it.27s_a_real_bug

But to answer to your question, currently these features cannot be turned off.

Elisa team, did you implement these yourself for Elisa, or are you making use of a feature of the Kirigami scrollview?
Comment 2 Matthieu Gallien 2018-04-16 07:21:21 UTC
We are only using the standard Qt Quick Controls v2 components.
Why would you want to disable smooth scrolling ?

It would really look odd to have a fluid UI except for scrolling that would jump magically from one position to another one.
Comment 3 Kushnir Bogdan 2018-04-17 13:24:08 UTC
(In reply to Nate Graham from comment #1)
> This is a bug tracker, not a user support hotline. See
> https://community.kde.org/Get_Involved/Bug_Reporting#Step_1:_Make_sure_it.
> 27s_a_real_bug
> 
> But to answer to your question, currently these features cannot be turned
> off.
> 
> Elisa team, did you implement these yourself for Elisa, or are you making
> use of a feature of the Kirigami scrollview?

Oh, sorry. I was supposing I add it to wishlist section.
Comment 4 Nate Graham 2018-04-17 13:38:13 UTC
I don't think it makes any sense to turn this off only for Elisa. It *maaaaaaybe* might make sense as a systemwide behavior change, but even then it seems dubious to me. Suggest WONTFIX.
Comment 5 Kushnir Bogdan 2018-04-17 13:50:30 UTC
(In reply to Matthieu Gallien from comment #2)
> Why would you want to disable smooth scrolling ?

Several reasons: 
1. It takes more time. Animation drawing needs time.
2. In Elisa it doesn't looks smooth on my laptop(intel_hd 4000).
3. I can't read the text while it's moving.
Comment 6 Nate Graham 2018-04-17 14:29:06 UTC
(In reply to Kushnir Bogdan from comment #5)
> (In reply to Matthieu Gallien from comment #2)
> > Why would you want to disable smooth scrolling ?
> 
> Several reasons: 
> 1. It takes more time. Animation drawing needs time.
> 2. In Elisa it doesn't looks smooth on my laptop(intel_hd 4000).

Huh, that graphics hardware should be easily sufficient. Do you see the same issue with other smooth scrolling implementations, or just in Elisa?

> 3. I can't read the text while it's moving.

Are we talking about the same thing? Smooth scrolling should make text *more* readable while moving, as it will animate its position rather than jumping from one position to another.
Comment 7 Kushnir Bogdan 2018-04-17 16:56:35 UTC
(In reply to Nate Graham from comment #6)

> Do you see the same issue with other smooth scrolling implementations, or just in Elisa?

No. Had checked on Firefox and Chromium recently. Also when scrolling in Elisa, it consume a lot of CPU(about 22% of i5-3360m)

> Are we talking about the same thing? Smooth scrolling should make text *more* readable while moving, as it will animate its position rather than jumping from one position to another.

Yes. But when scroll step isn't very huge(<10-15% of page) and you know scrolling directcion, our brain can imagine this smooth scrolling without any smooth scrolling in fact. So in my opinion, turning on/off smooth scrolling is has a place to be, like in Firefox. Or very fast scrolling animation speed with low CPU consumption.
Comment 8 Nate Graham 2018-04-17 22:41:39 UTC
Let's leave this open to track the scroll performance issue, which is the only Elisa-specific part of the initial report.
Comment 9 Matthieu Gallien 2018-06-04 16:02:50 UTC
Let's be clear about this report.
I will not remove smooth scrolling given this is one of the few main design points of Elisa that are not negotiable.
I will also not add any option to allow disabling it even if that would be feasible with the Qt Quick Controls.
If there is a performance problem, we need to track it down and solve it. I am unsure how one can do that independently of the GPU.
For example, I know that depending on the screen size and GPU, the performance can vary a lot with big animations in Elisa.
Comment 10 Matthieu Gallien 2018-06-04 16:03:02 UTC
Let's be clear about this report.
I will not remove smooth scrolling given this is one of the few main design points of Elisa that are not negotiable.
I will also not add any option to allow disabling it even if that would be feasible with the Qt Quick Controls.
If there is a performance problem, we need to track it down and solve it. I am unsure how one can do that independently of the GPU.
For example, I know that depending on the screen size and GPU, the performance can vary a lot with big animations in Elisa.
Comment 11 Matthieu Gallien 2018-09-05 19:26:14 UTC
I did again some test and it is hard to see if there is really a performance problem with smooth scrolling.