[KCM] Try harder to display all text in the visualizations
ClosedPublic

Authored by ngraham on Mon, Oct 21, 3:24 PM.

Details

Diff Detail

Repository
R104 KScreen
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham created this revision.Mon, Oct 21, 3:24 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMon, Oct 21, 3:24 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Mon, Oct 21, 3:24 PM

Looks good. If the display is too small to display two rows or the name is even longer, does it just elide?

One question I have long term is what information actually needs to be in there. Currently we have name and the logical size. Would only the name or name + resolution make more sense?

ngraham added a comment.EditedMon, Oct 21, 4:46 PM

Looks good. If the display is too small to display two rows or the name is even longer, does it just elide?

Yep.

One question I have long term is what information actually needs to be in there. Currently we have name and the logical size. Would only the name or name + resolution make more sense?

Unless I'm misunderstanding things, it's already is showing the name and the resolution. The name is mandatory I think; not so sure about the resolution.

You good with getting this into 5.17 now, and we can investigate what should be displayed in there for 5.18?

romangg accepted this revision.Mon, Oct 21, 5:18 PM

Looks good. If the display is too small to display two rows or the name is even longer, does it just elide?

Yep.

One question I have long term is what information actually needs to be in there. Currently we have name and the logical size. Would only the name or name + resolution make more sense?

Unless I'm misunderstanding things, it's already is showing the name and the resolution. The name is mandatory I think; not so sure about the resolution.

It shows the logical size. If you have a 3840x2160 resolution screen with scale factor 2, it shows 1920x1080. I guess it's this way only on Wayland

You good with getting this into 5.17 now, and we can investigate what should be displayed in there for 5.18?

Yes, 5.17.

This revision is now accepted and ready to land.Mon, Oct 21, 5:18 PM
romangg added a comment.EditedMon, Oct 21, 5:19 PM

Add a more meaningful summary please and replace the 5.17 text with it.

This revision was automatically updated to reflect the committed changes.

Add a more meaningful summary please and replace the 5.17 text with it.

Oops sorry, I missed this before landing it. :(