D10723: hide primary display combo when not supported

Sebastian Kügler noreply at phabricator.kde.org
Wed Feb 21 18:23:42 UTC 2018


sebas created this revision.
sebas added reviewers: Plasma, dvratil.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
sebas requested review of this revision.

REVISION SUMMARY
  Some backends, for example the kwayland backend, do not support
  setting the primary display. The backends already make their
  feature set known so we can just hide it from the UI.
  
  It's enough to hide it from the UI since setting it doesn't make
  any difference, the libkscreen backend will simply ignore it
  internally, so it's just honest to not show it.
  
  BUG:390638

TEST PLAN
  ran kscreen kcm on both, X11 and wayland platforms, combo
  and label are shown and hidden as expected

REPOSITORY
  R104 KScreen

BRANCH
  sebas/pcombo

REVISION DETAIL
  https://phabricator.kde.org/D10723

AFFECTED FILES
  kcm/src/widget.cpp
  kcm/src/widget.h

To: sebas, #plasma, dvratil
Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180221/34d23e34/attachment.html>


More information about the Plasma-devel mailing list