D12448: Fix KCompletionBox on wayland

[Anthony Fieroni]

anthonyfieroni added a comment.


  Why not
  
    static const auto flags = qGuiApp->platformName() == QLatin1String("wayland") ? Qt::WindowFlags(Qt::ToolTip | Qt::FramelessWindowHint | Qt::BypassWindowManagerHint) :
                                                                                    Qt::WindowFlags(Qt::Window | Qt::FramelessWindowHint | Qt::BypassWindowManagerHint);
    q->setWindowFlags(flags);
  
  It's not need to check in every completion construction.

REPOSITORY
  R284 KCompletion

BRANCH
  master

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

To: apol, #frameworks, davidedmundson
Cc: anthonyfieroni, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180422/f782c870/attachment.html>