Bug 478599

Summary: Misplaced "Positioning" combobox
Product: [Applications] systemsettings Reporter: Patrick Silva <bugseforuns>
Component: kcm_screenlockerAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal Keywords: qt6
Priority: NOR    
Version First Reported In: 5.90.0   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 6.0
Sentry Crash Report:
Attachments: screenshot

Description Patrick Silva 2023-12-16 10:01:18 UTC
Created attachment 164220 [details]
screenshot

STEPS TO REPRODUCE
1. open Screen Locking KCM
2. click on "Configure..." button
3. 

OBSERVED RESULT
bug seen in the attached screenshot

EXPECTED RESULT
combobox in the correct position

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.90.90
KDE Frameworks Version: 5.247.0
Qt Version: 6.6.1
Graphics Platform: Wayland
Comment 1 Bug Janitor Service 2023-12-18 07:35:24 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kscreenlocker/-/merge_requests/194
Comment 2 Bug Janitor Service 2023-12-18 07:41:41 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3694
Comment 3 fanzhuyifan 2023-12-18 08:43:23 UTC
Git commit ca61fe5249030acaf51478823ecfa3a323088834 by Yifan Zhu.
Committed on 18/12/2023 at 08:33.
Pushed by fanzhuyifan into branch 'master'.

kcm/ui: add required id and parentLayout property

For correct alignment with wallpaper config, an ancestor node must have
id "appearanceRoot" and property "parentLayout". See the files
plasma-workspace/wallpapers/image/
- imagepackage/contents/ui/config.qml
- slideshowpackage/contents/ui/SlideshowComponent.qml
FIXED-IN: 6.0

M  +3    -0    kcm/ui/Appearance.qml

https://invent.kde.org/plasma/kscreenlocker/-/commit/ca61fe5249030acaf51478823ecfa3a323088834
Comment 4 fanzhuyifan 2023-12-18 08:45:32 UTC
Git commit 0c55fc972641a3f306317519f3ca8ac6e5201832 by Yifan Zhu.
Committed on 18/12/2023 at 08:41.
Pushed by fanzhuyifan into branch 'master'.

wallpaper/image: add comments on usage requirement

For correct alignment with wallpaper config, an ancestor node must have
id "appearanceRoot" and property "parentLayout".

M  +3    -0    wallpapers/image/imagepackage/contents/ui/config.qml
M  +3    -0    wallpapers/image/slideshowpackage/contents/ui/SlideshowComponent.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/0c55fc972641a3f306317519f3ca8ac6e5201832