| Summary: | Better UX for loading of heavy Kirigami UI elements | ||
|---|---|---|---|
| Product: | [Applications] Discover | Reporter: | Alexander Mentyu <notuxius> |
| Component: | discover | Assignee: | Aleix Pol <aleixpol> |
| Status: | RESOLVED FIXED | ||
| Severity: | wishlist | CC: | aleixpol, notmart |
| Priority: | NOR | ||
| Version First Reported In: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Neon | ||
| OS: | Linux | ||
| Latest Commit: | https://commits.kde.org/discover/349be75c6cab67e389a54ae2c2ce4c893a5d87c1 | Version Fixed/Implemented In: | |
| Sentry Crash Report: | |||
| Attachments: | long-loading-of-content-in-discover | ||
|
Description
Alexander Mentyu
2017-08-20 08:59:04 UTC
Kirigami is already displaying busy indicator for downloadable content - the wish is to display busy indicator also upon loading of local or cached content - for example loading of 'Applications' section in Discover. Created attachment 107396 [details]
long-loading-of-content-in-discover
i think it's an issue of discover? is just the application that can know if a page can be potentially problematic to load Yes, I don't think we want Kirigami to try to be overly too smart here. Alexander, do you know why it takes you so long? I've never seen it that slow. If you could do some profiling there that would be very welcome. Git commit 349be75c6cab67e389a54ae2c2ce4c893a5d87c1 by Aleix Pol. Committed on 14/09/2017 at 15:03. Pushed by apol into branch 'Plasma/5.11'. Optimize adding resources to the model Don't reset the model when it starts if it's empty already anyway. When inserting elements, don't do the cherry-picked insertion unless there is already things on the view. M +7 -4 libdiscover/resources/ResourcesProxyModel.cpp https://commits.kde.org/discover/349be75c6cab67e389a54ae2c2ce4c893a5d87c1 |