Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dash menu: STILL get the issue of opening the menu every once in a while and hovering any entry (shutdown options, favs, programs) first icon does not highlight and therefore cant be clicked. #242

Open
star-buck opened this issue Sep 1, 2020 · 2 comments
Assignees

Comments

@star-buck
Copy link
Contributor

No description provided.

@star-buck
Copy link
Contributor Author

the menu on latest manajro is still laggy, even after hovering for a while over the categories, it takes another 1-2 seconds before any icon the mouse cursor is on appears to be hovered and selected....

@davidedmundson
Copy link

davidedmundson commented Oct 11, 2021

I did some profiling. It's spending way too much time loading icons for the main grid. This doesn't affect kickoff as much as there are fewer icons visible at a single time.
The slowdown is for a number of reasons, so this won't be fixed in a single commit. I will split out some of obvious waste spotted so far:

  • We're creating a new Plasma::Theme object each icon paint to get the palette (https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/349)

  • Icon delegates are not recylced

  • The "enabled" property gets propagated to the icons triggering both a redraw and a slow path painting effects
    The "all apps grid" and "mainGrid" both exist at once and toggle enabled and visibility.
    https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/598

  • The logic in onPositionChanged (DashboardRepresentation.qml:885) looks very weird. Sometimes we do things immediately, sometimes after a 50ms timeout. The immediate call gets triggered too often, and the compressed path (with the timeout) is waaay too long.

I will make commits for them, and then we'll see where things stand.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants