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

[Méven Car]

meven added a comment.


  In D19535#432739 <https://phabricator.kde.org/D19535#432739>, @elvisangelaccio wrote:
  
  > In D19535#429447 <https://phabricator.kde.org/D19535#429447>, @meven wrote:
  >
  > > In D19535#428339 <https://phabricator.kde.org/D19535#428339>, @elvisangelaccio wrote:
  > >
  > > > This is not a KWin bug. `QCursor::pos()}` is not reliable on Wayland and should not be used.
  > >
  > >
  > > Could we document this ? So that the community avoids discovering a horde a bugs once wayland usage grows. Maybe on https://community.kde.org/Plasma/Wayland_Showstoppers
  >
  >
  > Better on https://community.kde.org/Guidelines_and_HOWTOs/Wayland_Porting_Notes (where we already explain how to avoid similar bugs).
  
  
  I've added a note about context Menus and QCursor::pos() :
  https://community.kde.org/Guidelines_and_HOWTOs/Wayland_Porting_Notes#Context_Menu
  Thanks :)

REPOSITORY
  R318 Dolphin

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

To: meven, #dolphin, elvisangelaccio
Cc: elvisangelaccio, 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/20190317/7428bef1/attachment.htm>