D10197: Fix krunner's alt+f2 on wayland
    David Edmundson 
    noreply at phabricator.kde.org
       
    Wed Jan 31 00:21:15 UTC 2018
    
    
  
davidedmundson accepted this revision.
davidedmundson added a comment.
This revision is now accepted and ready to land.
  For context, krunner inherits from Dialog. 
  Dialog does all this stuff.  Duplicating it here is wrong.
  
  Must have been ported first or something.
INLINE COMMENTS
> view.cpp:194-200
>      bool setState = event->type() == QEvent::Show;
>      if (event->type() == QEvent::PlatformSurface) {
>          setState = (static_cast<QPlatformSurfaceEvent*>(event)->surfaceEventType() == QPlatformSurfaceEvent::SurfaceCreated);
>      }
>      if (setState) {
>          KWindowSystem::setState(winId(), NET::SkipTaskbar | NET::SkipPager);
>      }
This relates to the QXcbWindow comment that you removed, can you leave it.
(I /think/ you can remove this code, it relates to David Rosca's patch that landed ages ago. Check first though)
REPOSITORY
  R120 Plasma Workspace
BRANCH
  master
REVISION DETAIL
  https://phabricator.kde.org/D10197
To: apol, #plasma, davidedmundson
Cc: ngraham, davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180131/7d95af47/attachment.html>
    
    
More information about the Plasma-devel
mailing list