Bug 412908 - scrolling of app description page is slow/choppy on Wayland
Summary: scrolling of app description page is slow/choppy on Wayland
Status: RESOLVED FIXED
Alias: None
Product: Discover
Classification: Applications
Component: discover (show other bugs)
Version: unspecified
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Dan Leinir Turthra Jensen
URL:
Keywords: wayland
Depends on:
Blocks:
 
Reported: 2019-10-13 14:49 UTC by Patrick Silva
Modified: 2019-11-07 13:45 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.17.3


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2019-10-13 14:49:47 UTC
STEPS TO REPRODUCE
1. open discover on Wayland session
2. search for an app with many reviews (gimp, for example)
3. click on "gimp" search result to open its description page
4. click on "show all 138 reviews..." link
5. scroll the reviews list using touchpad, mouse or touchscreen

OBSERVED RESULT
scrolling is slow/choppy

EXPECTED RESULT
normal scrolling

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.17.80
KDE Frameworks Version: 5.64.0
Qt Version: 5.13.1
Comment 1 Patrick Silva 2019-10-13 16:57:05 UTC
scrolling through search results is also slow/choppy.
Search for "player" and try to scroll the results list...
Comment 2 Aleix Pol 2019-10-28 18:53:17 UTC
Why do you think it's because of wayland?
Comment 3 Patrick Silva 2019-10-28 19:03:34 UTC
(In reply to Aleix Pol from comment #2)
> Why do you think it's because of wayland?

cannot reproduce on X11.
Comment 4 Rainer Finke 2019-11-03 08:48:14 UTC
Just experienced the same issue. Discover on Wayland is really slow, I just need to click on "Plasma Extensions" and then it is only usable with an extreme lag. Just scroll in the app list or type in word in the search field to experience the lag. But there is no real CPU spike.

Operating System: Arch Linux 
KDE Plasma Version: 5.17.2
KDE Frameworks Version: 5.63.0
Qt Version: 5.14.0 beta2
Kernel Version: 5.3.8-arch1-1
OS Type: 64-bit
Comment 5 joern477 2019-11-04 19:03:33 UTC
I can report the same issue here. Scrolling is really ugly on wayland and fine on X11.

... While writing this I played a bit around in Discover and suddenly the scrolling started working fine even on wayland. I've done a complete reboot and scrolling is still fine.

Maybe discover is doing something in the background which stopped at the point when scrolling gets fine? I mostly open discover when updates are displayed directly after boot.
Will have a look how scrolling works when updates are available again the next days.

Operating System: KDE neon 5.17
KDE Plasma Version: 5.17.2
KDE Frameworks Version: 5.63.0
Qt Version: 5.13.1
Kernel Version: 4.15.0-66-generic
OS Type: 64-bit
Processors: 4 × Intel® Core™ i5-5300U CPU @ 2.30GHz
Memory: 15,1 GiB of RAM
Comment 6 joern477 2019-11-05 19:10:32 UTC
Checked with the updates today. I don't see any relation to updates anymore.

But I observed the following:

- "Applications" section scrolls perfectly fine for me
    -> Icons seems to be pre-loaded, they are directly there while scrolling
- "Application Addons" and "Plasma Addons" scrolls really bad
    -> Icons seems to be loaded while scrolling and are updated while scrolling
Comment 7 joern477 2019-11-05 19:22:11 UTC
"Applications" seem to have dedicated Icons in the list.
Everything I do in this section is super fast.

"Application Addons" and "Plasma Addons" instead use the first Image as Icon in the list. 
Not only the list is sluggish. Everything for this to sections is sluggish for me. Selecting an app (highlight effect from one list item to the next takes really long) clicking an addon and loading its information is slow. And also image browsing for addons is sluggish.
Comment 8 Aleix Pol 2019-11-07 02:37:50 UTC
Git commit 836b5595516c323ad75b27ecbec11f7fdbd857ca by Aleix Pol.
Committed on 07/11/2019 at 02:36.
Pushed by apol into branch 'Plasma/5.17'.

ui: improve busy indicator handling

Simplifies the logic, makes sure it's not moving unless we are seeing
it.

M  +7    -51   discover/qml/ApplicationsListPage.qml

https://commits.kde.org/discover/836b5595516c323ad75b27ecbec11f7fdbd857ca