Bug 460397

Summary: Scale text may not fit into display view
Product: [Applications] systemsettings Reporter: gudvinr+kde
Component: kcm_kscreenAssignee: kscreen-bugs-null <kscreen-bugs-null>
Status: RESOLVED FIXED    
Severity: minor CC: nate, plasma-bugs-null
Priority: NOR Keywords: usability
Version First Reported In: 5.26.0   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 5.27.6
Sentry Crash Report:
Attachments: screenshot

Description gudvinr+kde 2022-10-13 22:47:40 UTC
Created attachment 152792 [details]
screenshot

SUMMARY

If you change scale and your display preview is too small or if display is rotated, part of its description is being cut off.

STEPS TO REPRODUCE
1. Open Display Configuration
2. Change scale and rotate monitor so description text doesn't fit horizontally

OBSERVED RESULT
As seen on screenshot, 115% is not being shown.

EXPECTED RESULT
Whole thing is visible and can be accessed.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux
KDE Plasma Version: 5.26.0
KDE Frameworks Version: 5.99.0
Qt Version: 5.15.6
Kernel Version: 6.0.1-zen1-1-zen (64-bit)
Graphics Platform: Wayland
Comment 1 Nate Graham 2022-10-14 19:21:18 UTC
I guess we should wrap.
Comment 2 Nate Graham 2022-10-14 19:22:13 UTC
Hah, indeed we should. The other label wraps, so the fact that this one doesn't is just a bug. Will fix.
Comment 3 Nate Graham 2022-10-14 19:23:55 UTC
Git commit 1c5b7ae45eb9f464cb79269c3e97ca96b5351ea4 by Nate Graham.
Committed on 14/10/2022 at 19:22.
Pushed by ngraham into branch 'master'.

KCM: wrap resolution label too

We wrap the name label, so we should wrap the resolution label too. It's
better than eliding, at least.
FIXED-IN: 5.26.1

M  +1    -0    kcm/package/contents/ui/Output.qml

https://invent.kde.org/plasma/kscreen/commit/1c5b7ae45eb9f464cb79269c3e97ca96b5351ea4
Comment 4 Nate Graham 2022-10-14 19:24:21 UTC
Git commit fdbf6f8a006ebb7376927f02013f8c91bcdaba35 by Nate Graham.
Committed on 14/10/2022 at 19:24.
Pushed by ngraham into branch 'Plasma/5.26'.

KCM: wrap resolution label too

We wrap the name label, so we should wrap the resolution label too. It's
better than eliding, at least.
FIXED-IN: 5.26.1
(cherry picked from commit 1c5b7ae45eb9f464cb79269c3e97ca96b5351ea4)

M  +1    -0    kcm/package/contents/ui/Output.qml

https://invent.kde.org/plasma/kscreen/commit/fdbf6f8a006ebb7376927f02013f8c91bcdaba35
Comment 5 gudvinr+kde 2022-11-18 18:29:57 UTC
Nate, is it possible to use something like Unicode ZWSP character around @ so it can break at proper boundaries? Right now it shows like this:

2560x1440@11
5%
Comment 6 Nate Graham 2022-11-18 20:48:22 UTC
Seems possible. Please feel free to submit a merge request for that, as it's probably fairly easy to do. If you don't feel up to it, please open a new bug report to track that, as it's a separate enhancement. Please use the "wishlist" severity for it, if you choose to go down this path. Thanks!
Comment 7 Nate Graham 2023-06-15 21:08:13 UTC
Git commit 5cf490f4b2c2532c236a6868a8524996f62d0328 by Nate Graham, on behalf of Maxim S.
Committed on 15/06/2023 at 21:08.
Pushed by ngraham into branch 'master'.

Add boundaries to resolution text

The text of the resolution with scale breaks in the middle of the number.

I am adding ZWS (U+200B) that acts like word boundary so text can
wrap nicely if there's not enough space.
FIXED-IN: 5.27.6

M  +1    -1    kcm/ui/Output.qml

https://invent.kde.org/plasma/kscreen/-/commit/5cf490f4b2c2532c236a6868a8524996f62d0328
Comment 8 Nate Graham 2023-06-15 21:09:07 UTC
Git commit 4614c0e72889f5e63f5e09a0aebd289630ed7634 by Nate Graham, on behalf of Maxim S.
Committed on 15/06/2023 at 21:09.
Pushed by ngraham into branch 'Plasma/5.27'.

Add boundaries to resolution text

The text of the resolution with scale breaks in the middle of the number.

I am adding ZWS (U+200B) that acts like word boundary so text can
wrap nicely if there's not enough space.
FIXED-IN: 5.27.6


(cherry picked from commit 5cf490f4b2c2532c236a6868a8524996f62d0328)

M  +1    -1    kcm/package/contents/ui/Output.qml

https://invent.kde.org/plasma/kscreen/-/commit/4614c0e72889f5e63f5e09a0aebd289630ed7634