Bug 422871

Summary:	Positions of sensors dragged in widget settings are remembered even if the settings window is canceled
Product:	[Plasma] plasmashell	Reporter:	Patrick Silva <bugseforuns>
Component:	System Monitor widgets	Assignee:	Marco Martin <notmart>
Status:	RESOLVED FIXED
Severity:	normal	CC:	plasma-bugs-null
Priority:	NOR
Version First Reported In:	master
Target Milestone:	1.0
Platform:	Neon
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/libksysguard/commit/ca980e7297868956e668d65e91fec58029667282	Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	screen recording

Description Patrick Silva 2020-06-12 13:20:28 UTC

Created attachment 129260 [details]
screen recording

SUMMARY
Watch the attached screen recording please.
The sensors are listed in their correct positions in settings window after Plasma to be restarted.

EXPECTED RESULT
reopened settings window should not show any discarted change

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.19.80
KDE Frameworks Version: 5.71.0
Qt Version: 5.14.2

Comment 1 Marco Martin 2020-06-18 09:01:47 UTC

Git commit ca980e7297868956e668d65e91fec58029667282 by Marco Martin.
Committed on 18/06/2020 at 09:01.
Pushed by mart into branch 'Plasma/5.19'.

reset the page when reloaded

when the dialog gets closed the page parent changes to null
as well when the active config pages is switched.
when this appen, reload the config

M  +9    -0    faces/ConfigSensors.qml

https://invent.kde.org/plasma/libksysguard/commit/ca980e7297868956e668d65e91fec58029667282