[Differential] [Reopened] D4113: Correct inital loading of BorderActivate

[David Edmundson]

davidedmundson reopened this revision.
davidedmundson added a comment.
This revision is now accepted and ready to land.


  > I wonder what happens if we pass "-1" and what is " " cast as...
  
  If you pass -1, then registerScreenEdge would reject it, and " " would cast as 0. 
  So absolutely worst case you get an extra border, nothing too bad.
  
  However, you're right that validation on the JS side would be cleaner. Will do that.

REPOSITORY
  R108 KWin

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

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

To: davidedmundson, #plasma, mart
Cc: luebking, graesslin, mart, plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170117/8f19ef1c/attachment.html>