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