D5748: workaround for correct wayland positioning

[David Edmundson]

davidedmundson added a comment.


  Is it tested on X, Qt 5.7 as well as Qt 5.9
  
  If you want me to do any of those, just ask.

INLINE COMMENTS

> view.cpp:250
> +    } else if (event->type() == QEvent::Hide) {
> +        delete m_plasmaShellSurface;
>      } else if (m_plasmaShellSurface && event->type() == QEvent::Move) {

we need  m_plasmaShellSurface = nullptr;

this leaves a dangling pointer,  which is both dangerous but also means we won't re-create it in ExposeEvent, I would have assumed that would mean the second show won't work.

REPOSITORY
  R120 Plasma Workspace

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

To: mart, #plasma, #plasma_on_wayland
Cc: davidedmundson, graesslin, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, eliasp, sebas, apol, hein, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170516/39b6ab36/attachment.html>