Bug 465318 - The "Orientation" label is sometimes inappropriately set to be the buddy for the orientation row even when automatic orientation is supported and its UI is visible
Summary: The "Orientation" label is sometimes inappropriately set to be the buddy for ...
Status: RESOLVED WORKSFORME
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_kscreen (other bugs)
Version First Reported In: 5.26.90
Platform: openSUSE Linux
: NOR minor
Target Milestone: ---
Assignee: kscreen-bugs-null@kde.org
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-02-05 13:45 UTC by Ilya Bizyaev
Modified: 2025-11-09 03:47 UTC (History)
6 users (show)

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

Attachments
Screenshot (109.54 KB, image/png)
2023-02-05 13:45 UTC, Ilya Bizyaev 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Ilya Bizyaev 2023-02-05 13:45:46 UTC 
Created attachment 155959 [details]
Screenshot

SUMMARY

The "Orientation" label should be displayed next to the "Automatic" checkbox, otherwise it looks like the choice of automatic vs manual applies to the scaling instead.


SOFTWARE/OS VERSIONS
Operating System: openSUSE Leap 15.4
KDE Plasma Version: 5.26.90
KDE Frameworks Version: 5.102.0
Qt Version: 5.15.8
Kernel Version: 5.14.21-150400.24.41-default (64-bit)
Graphics Platform: Wayland

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2023-02-07 17:51:38 UTC 
That's weird, it's in the right place for me with git master on Wayland, which isn't too far off from what you've got. Let me inspect the code to see what might be going on.
Comment 2 Nate Graham 2023-02-07 17:56:47 UTC 
The code has this:

Kirigami.FormData.label: i18n("Orientation:")
Kirigami.FormData.buddyFor: autoRotateColumn.visible ? autoRotateRadio : orientation

So what appears to be going on is that it's being set as the buddy for the "orientation" row, rather than "autoRotateRadio". But that doesn't make any sense since this should only happen when "autoRotateColumn" isn't visible, but your screenshot clearly shows that it is visible.

Maybe it gets set to visible too late and the binding on Kirigami.FormData.buddyFor doesn't re-evaluate in Kirigami.FormLayout?
Comment 3 Ilya Bizyaev 2024-03-16 17:01:52 UTC 
I tried to test this with Plasma 6.0 (Wayland), but the automatic rotation setting seems to have vanished. Has it been moved?
`iio-sensor-proxy` is installed.
Comment 4 Nate Graham 2024-03-18 18:30:22 UTC 
It's there for me, and it's also in the right place. I suspect that the issue you're seeing now is connected to the earlier one with the label appearing in the wrong place.
Comment 5 Nate Graham 2025-10-10 17:47:32 UTC 
Pretty sure this was fixed at some point, and auto-rotation options also not appearing properly was also fixed recently. Any chance you can test this with git master, or with 6.5.0 when it's released in 11 days?
Comment 6 Bug Janitor Service 2025-10-25 03:47:39 UTC 
🐛🧹 ⚠️ This bug has been in NEEDSINFO status with no change for at least 15 days. Please provide the requested information, then set the bug status to REPORTED. If there is no change for at least 30 days, it will be automatically closed as RESOLVED WORKSFORME.

For more information about our bug triaging procedures, please read https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging.

Thank you for helping us make KDE software even better for everyone!
Comment 7 Bug Janitor Service 2025-11-09 03:47:50 UTC 
🐛🧹 This bug has been in NEEDSINFO status with no change for at least 30 days. Closing as RESOLVED WORKSFORME.