[Differential] [Commented On] D3006: fix screenForContainment when screen ids > count

[davidedmundson (David Edmundson)]

davidedmundson added a comment.


  I can see how this is a bug that needs fixing for the a screen ID > screenCount but what about the case where the screenID isn't valid anymore?
  
  i.e
  a panel was on screen 2 you shutdown (assuming ID's are also 1 and 2 respectively)
  when you boot up you now only have 1 screen
  
  Before the code would return -1
  This would put it on screen 2 which doesn't exist - will that cause a problem?

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: mart, #plasma
Cc: davidedmundson, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161010/f7180958/attachment.html>