D8444: Define default Action when we plug unknown monitor

Sebastian Kügler noreply at phabricator.kde.org
Tue Oct 31 09:00:58 UTC 2017


sebas added a comment.


  Once you're done with the code, please also add screenshot so usability can have a look at it. I'm not sure we should add UI for this, the screen setup is already very complex and hard to understand. Perhaps a set of clear use-cases would make this clearer?
  
  No unit tests for the code (I know, hard to do) will add to the maintenance burden, which is already high.
  
  Just seeing the code is definitely not enough for me to ACK this.

REPOSITORY
  R104 KScreen

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

To: mlaurent, davidedmundson
Cc: sebas, graesslin, dvratil, davidedmundson, ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171031/0c620e8c/attachment.html>


More information about the Plasma-devel mailing list