Bug 462775 - App details CPU usage should follow the settings of the general page
Summary: App details CPU usage should follow the settings of the general page
Status: RESOLVED FIXED
Alias: None
Product: plasma-systemmonitor
Classification: Applications
Component: general (show other bugs)
Version: 5.26.4
Platform: Neon Linux
: NOR wishlist
Target Milestone: ---
Assignee: KSysGuard Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-12-08 10:22 UTC by Riccardo Robecchi
Modified: 2023-01-13 21:44 UTC (History)
5 users (show)

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


Attachments
Screenshot of the issue (36.37 KB, image/png)
2022-12-08 10:22 UTC, Riccardo Robecchi
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Riccardo Robecchi 2022-12-08 10:22:12 UTC
Created attachment 154421 [details]
Screenshot of the issue

SUMMARY
If you set the CPU usage of the general "Applications" tab to "text only" or "line chart", so without the scaling to 100%, the "details" pane does not follow this setting and displays data scaled to 100%, leading to a clear mismatch that can generate confusion.

STEPS TO REPRODUCE
1. Set CPU usage in the general "Applications" tab to "text only" or "line chart"
2. Open the details sidebar

OBSERVED RESULT
The details sidebar scales the data to 100%, irrespective of the user's choice for the general column.

EXPECTED RESULT
The details sidebar reflects the setting of the general column.

SOFTWARE/OS VERSIONS
Linux: KDE neon
KDE Plasma Version: 5.26.4
KDE Frameworks Version: 5.100.0
Qt Version: 5.15.7

ADDITIONAL INFORMATION
Comment 1 Bug Janitor Service 2023-01-13 10:07:29 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/199
Comment 2 Arjen Hiemstra 2023-01-13 14:28:37 UTC
Git commit 04de5c40e6fa999da713f7f1397814b7c55b2647 by Arjen Hiemstra.
Committed on 13/01/2023 at 11:50.
Pushed by ahiemstra into branch 'master'.

applicationstable: Use the same scaling for CPU details as the column display

Otherwise the two are not in sync which can lead to unexpected results.

M  +28   -2    src/faces/applicationstable/contents/ui/ApplicationDetails.qml
M  +2    -0    src/faces/applicationstable/contents/ui/ApplicationInformation.qml
M  +1    -0    src/faces/applicationstable/contents/ui/ApplicationsTableView.qml
M  +1    -0    src/table/ColumnDisplayModel.h

https://invent.kde.org/plasma/plasma-systemmonitor/commit/04de5c40e6fa999da713f7f1397814b7c55b2647