Bug 480160

Summary: "Import Page" doesn't import widgets.
Product: [Applications] plasma-systemmonitor Reporter: Stanislav Chernichkin <schernichkin>
Component: generalAssignee: KSysGuard Developers <ksysguard-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: ahiemstra, nate, plasma-bugs
Priority: NOR    
Version: 5.27.10   
Target Milestone: ---   
Platform: Manjaro   
OS: Linux   
Latest Commit: https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/dec9fb3a5582c70a361dbb8c6156f58d6debfefb Version Fixed In: 6.0

Description Stanislav Chernichkin 2024-01-22 11:12:51 UTC 
SUMMARY
***
"Import Page" doesn't import widgets added in system monitor UI.
***

STEPS TO REPRODUCE
1. Create new page in System Monitor.
2. Add some widgets.
3. Press "Save Changes" and then "Export Current Page".
4. Press "Import Page" and select page you have exported.

OBSERVED RESULT
Page loaded, but semi-empty, no widgets, just empty panes.

EXPECTED RESULT
Page should include all controls added to it before export.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 6.7.0.0-MANJARO
(available in About System)
KDE Plasma Version: 5.27.10
KDE Frameworks Version: 5.113.0
Qt Version: 5.15.12

ADDITIONAL INFORMATION
Comment 1 Bug Janitor Service 2024-02-01 14:30:37 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/250
Comment 2 Arjen Hiemstra 2024-02-02 12:07:28 UTC 
Git commit dc97c9719ccfeffbbfb6a8177ef7a0135361137f by Arjen Hiemstra.
Committed on 02/02/2024 at 12:01.
Pushed by ahiemstra into branch 'master'.

Make face cache entries unique per-page

Otherwise we risk pages ending up with empty sections because the
face is in another page.

M  +5    -3    src/page/FaceLoader.cpp

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/dc97c9719ccfeffbbfb6a8177ef7a0135361137f
Comment 3 Arjen Hiemstra 2024-02-02 12:09:28 UTC 
Git commit dec9fb3a5582c70a361dbb8c6156f58d6debfefb by Arjen Hiemstra.
Committed on 02/02/2024 at 12:08.
Pushed by ahiemstra into branch 'Plasma/6.0'.

Make face cache entries unique per-page

Otherwise we risk pages ending up with empty sections because the
face is in another page.


(cherry picked from commit dc97c9719ccfeffbbfb6a8177ef7a0135361137f)

M  +5    -3    src/page/FaceLoader.cpp

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/dec9fb3a5582c70a361dbb8c6156f58d6debfefb