D19535: Correctly position context menu of the information panel under wayland with a secondary screen

[Méven Car]

meven added a comment.


  I am not sure this is a KWin bug.
  
  The QCursor::pos() according to the documentation :
  
  > Returns the position of the cursor (hot spot) of the primary screen in global screen coordinates.
  
  https://doc.qt.io/qt-5/qcursor.html#pos
  
  But in a multi-screen context, this should not be used because it can't handle secondary screen as documented, which is the case here.
  If that is the problem's root it also means that any use of QCursor::pos() will cause a bug in multi-screen contexts.
  What do you think @ngraham #kwin <https://phabricator.kde.org/tag/kwin/> ?

REPOSITORY
  R318 Dolphin

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

To: meven, #dolphin
Cc: ngraham, nicolasfella, kfm-devel, alexde, feverfew, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190307/49842b36/attachment.htm>