Bug 505010 - Performance: LegendView is updating whilst in the process tab
Summary: Performance: LegendView is updating whilst in the process tab
Status: RESOLVED FIXED
Alias: None
Product: plasma-systemmonitor
Classification: Applications
Component: general (other bugs)
Version First Reported In: 6.4.80
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: KSysGuard Developers
URL:
Keywords: efficiency-and-performance
Depends on:
Blocks:
 
Reported: 2025-05-30 12:23 UTC by David Edmundson
Modified: 2025-10-03 21:53 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.19
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Edmundson 2025-05-30 12:23:35 UTC
Running hotspot whilst in the process view (with all columns turned off) showed 20% of time in a Legend updating. I assume this is the legend on the homepage? We want to disable things when they're not visible.
Comment 1 Nate Graham 2025-05-30 18:08:13 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.
Comment 2 Arjen Hiemstra 2025-09-24 15:56:36 UTC
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.
Comment 3 Arjen Hiemstra 2025-10-03 11:12:48 UTC
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