From 7e5dff648f0193df533066312597d028e5488795 Mon Sep 17 00:00:00 2001 From: Kai Uwe Broulik Date: Tue, 6 Mar 2018 14:42:08 +0100 Subject: [PATCH] [Dialog] Remove now obsolete KWindowSystem::setState calls The comment says it's resolved in Qt 5.6.1+ but Frameworks depends on Qt 5.7 nowadays Differential Revision: https://phabricator.kde.org/D10769 --- src/plasmaquick/dialog.cpp | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/src/plasmaquick/dialog.cpp b/src/plasmaquick/dialog.cpp index a7bcd35fc..02a8e4b05 100644 --- a/src/plasmaquick/dialog.cpp +++ b/src/plasmaquick/dialog.cpp @@ -1319,11 +1319,6 @@ void Dialog::componentComplete() { d->componentComplete = true; QQuickWindow::setVisible(d->visible); - if (d->visible) { - // FIXME TODO: We can remove this once we depend on Qt 5.6.1+. - // See: https://bugreports.qt.io/browse/QTBUG-26978 - KWindowSystem::setState(winId(), NET::SkipTaskbar | NET::SkipPager); - } d->updateTheme(); } @@ -1371,11 +1366,6 @@ void Dialog::setVisible(bool visible) d->mainItem->setVisible(true); } QQuickWindow::setVisible(visible); - if (visible) { - // FIXME TODO: We can remove this once we depend on Qt 5.6.1+. - // See: https://bugreports.qt.io/browse/QTBUG-26978 - KWindowSystem::setState(winId(), NET::SkipTaskbar | NET::SkipPager); - } //signal will be emitted and proxied from the QQuickWindow code } else { emit visibleChangedProxy();