| Summary: | Performance: LegendView is updating whilst in the process tab | ||
|---|---|---|---|
| Product: | [Applications] plasma-systemmonitor | Reporter: | David Edmundson <kde> |
| Component: | general | Assignee: | KSysGuard Developers <ksysguard-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | ahiemstra, nate, plasma-bugs-null |
| Priority: | NOR | Keywords: | efficiency-and-performance |
| Version First Reported In: | 6.4.80 | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/frameworks/kquickcharts/-/commit/9a7585f68ba077e8c281f3f20532d86967c4f1d3 | Version Fixed In: | 6.19 |
| Sentry Crash Report: | |||
|
Description
David Edmundson
2025-05-30 12:23:35 UTC
IIRC we wanted to keep it collecting data in the background, so that when you switch to the History page, it's instantly up-to-date. But for sure we don't need to draw anything visible on the graph while on another page. It'll be the history page, as indeed that gets loaded in the background to collect data. That includes all its charts and the models supporting those. They do not currently have visible/enabled checks, though some of them probably should. Git commit 9a7585f68ba077e8c281f3f20532d86967c4f1d3 by Arjen Hiemstra. Committed on 02/10/2025 at 12:43. Pushed by ahiemstra into branch 'master'. controls: Don't perform a layout in LegendLayout if we're invisible Doing a layout while we're invisible isn't very useful as no one will see it anyway. So just do an early return if the LegendLayout is not visible. M +1 -1 controls/LegendLayout.cpp https://invent.kde.org/frameworks/kquickcharts/-/commit/9a7585f68ba077e8c281f3f20532d86967c4f1d3 |