Bug 428724 - Freeze when changing the pie chart to a line chart on a new page
Summary: Freeze when changing the pie chart to a line chart on a new page
Status: RESOLVED FIXED
Alias: None
Product: plasma-systemmonitor
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: KSysGuard Developers
URL:
Keywords:
: 428908 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-11-05 13:14 UTC by Noah Davis
Modified: 2021-02-09 13:49 UTC (History)
4 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Noah Davis 2020-11-05 13:14:58 UTC
SUMMARY
After creating a new page, it's possible to freeze plasma-systemmonitor when changing the default pie chart to a line chart. No other changes were made.

STEPS TO REPRODUCE
1. Create a new page
2. edit the pie chart
3. change the pie chart to a line chart

OBSERVED RESULT
plasma-systemmonitor freezes and has to be killed

EXPECTED RESULT
plasma-systemmonitor should change the pie chart to a line chart without freezing

SOFTWARE/OS VERSIONS
Operating System: openSUSE Tumbleweed 20201030
KDE Plasma Version: 5.20.80
KDE Frameworks Version: 5.76.0
Qt Version: 5.15.1
Kernel Version: 5.8.4-1-default
OS Type: 64-bit
Processors: 4 × Intel® Core™ i7-6500U CPU @ 2.50GHz
Memory: 7.6 GiB of RAM
Graphics Processor: Mesa DRI Intel® HD Graphics 520
Comment 1 Justin Zobel 2020-11-06 09:41:55 UTC
I'm unable to reproduce this on plasma-systemmonitor from git master.

If you run it from a terminal does it give you any output that might indicate why it froze?
Comment 2 Arjen Hiemstra 2020-11-06 09:43:33 UTC
Git commit bf8722d56552ed5f558c01f4f6417d7928ce56a3 by Arjen Hiemstra.
Committed on 06/11/2020 at 09:43.
Pushed by ahiemstra into branch 'master'.

Do not use fillWidth on the face control on the face config page

It causes a layout loop that freezes plasma-systemmonitor

M  +2    -1    src/page/FaceConfigurationPage.qml

https://invent.kde.org/plasma/plasma-systemmonitor/commit/bf8722d56552ed5f558c01f4f6417d7928ce56a3
Comment 3 Arjen Hiemstra 2021-02-09 13:49:17 UTC
*** Bug 428908 has been marked as a duplicate of this bug. ***