[Differential] [Request, 37 lines] D4347: Avoid jumpy UI in Kicker Dialogs

[David Edmundson]

davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Under X11 we would see move events as two distinct jumpy frames.
  The Dialog would move, and then resize.
  
  This patch defers the move event until the same xcb flush as when we
  update the NET_WM_SYNC_COUNTER with the new window. This means we move
  and do the resize in the same event.

TEST PLAN
  Ran under kwin, all buttery smooth
  Ran under openbox, no better, but also no worse
  Menu still appeared in the same place, and would still be restricted to fit on screen.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

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

AFFECTED FILES
  applets/kicker/plugin/submenu.cpp
  applets/kicker/plugin/submenu.h

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

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