Bug 413014

Summary: KScreen design degradation in Plasma 5.17
Product: [Plasma] KScreen Reporter: voidfragment <itnativa>
Component: kcmAssignee: kscreen-bugs-null <kscreen-bugs-null>
Status: RESOLVED FIXED    
Severity: wishlist CC: nate, subdiff
Priority: NOR    
Version: 5.17.0   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In: 5.17.1
Attachments: new kscreen design

Description voidfragment 2019-10-16 04:39:00 UTC
Created attachment 123227 [details]
new kscreen design

Guys, the new design of kscreen is incorrect. Replacement the evident picture of a second monitor by the hardly perceptible selector is a complete failure. 

Please, revert it.
Comment 1 Roman Gilg 2019-10-16 11:26:21 UTC
Relax dude. Showing the disabled monitor in the overall view is not a nice solution since a disabled monitor should not take space away since it doesn't in reality. Imagine you have three monitors and disable only one of them.

You got some proposal how it should be done design-wise instead? I was thinking of adding monitor symbols to one of the sides of the view for each disabled output. You think that would be good or not?
Comment 2 Nate Graham 2019-10-16 22:09:19 UTC
Replacing the tiny dots with a combobox should fix this: https://phabricator.kde.org/D24687
Comment 3 voidfragment 2019-10-17 05:11:20 UTC
(In reply to Nate Graham from comment #2)
> Replacing the tiny dots with a combobox should fix this:
> https://phabricator.kde.org/D24687

Thank you!
Comment 4 Kai Uwe Broulik 2019-10-17 13:36:13 UTC
Git commit 7811411c6425dd52bcd732a5910d177fee4a89d2 by Kai Uwe Broulik.
Committed on 17/10/2019 at 13:35.
Pushed by broulik into branch 'Plasma/5.17'.

[KCM] Add output selector ComboBox

Since hidden outputs aren't shown in the output view for some reason, provide a ComboBox to switch between outputs
since the bullet points with swipe view aren't very obvious.

Differential Revision: https://phabricator.kde.org/D24687

M  +0    -8    kcm/package/contents/ui/OutputPanel.qml
M  +26   -1    kcm/package/contents/ui/Panel.qml

https://commits.kde.org/kscreen/7811411c6425dd52bcd732a5910d177fee4a89d2