From 7967cb4f4860df436dc2e7e21c4eaca6a9b05f70 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Samuel=20R=C3=B8dal?= Date: Wed, 13 Jun 2012 17:05:52 +0200 Subject: Prevent ending up in a state where focus is perpetually grabbed. Mouse / enter / leave / key events etc are all blocked when a window has the blockedByModalWindow flag set. The problem appears if a QWindow is created and only later directly or indirectly parented to a modal window that's currently showing. Since the decision on whether a window should be blocked or not is based on its parent / transient parent chain, we need to reevaluate the blocked status each time the parent or transient parent of a window changes. Task-number: QTBUG-26112 Change-Id: Ida6b118b556fe26d17fa86335a0fe7baddc7eaf8 Reviewed-by: Lars Knoll Reviewed-by: Bradley T. Hughes --- src/gui/kernel/qguiapplication.cpp | 30 ++++++++++++++++++++---------- 1 file changed, 20 insertions(+), 10 deletions(-) (limited to 'src/gui/kernel/qguiapplication.cpp') diff --git a/src/gui/kernel/qguiapplication.cpp b/src/gui/kernel/qguiapplication.cpp index 3cd8d5f800..e22ccc762d 100644 --- a/src/gui/kernel/qguiapplication.cpp +++ b/src/gui/kernel/qguiapplication.cpp @@ -402,18 +402,30 @@ QWindow *QGuiApplication::modalWindow() return QGuiApplicationPrivate::self->modalWindowList.first(); } -void QGuiApplicationPrivate::showModalWindow(QWindow *window) +void QGuiApplicationPrivate::updateBlockedStatus(QWindow *window) { - self->modalWindowList.prepend(window); + bool shouldBeBlocked = false; + if (window->windowType() != Qt::Tool && !self->modalWindowList.isEmpty()) + shouldBeBlocked = self->isWindowBlocked(window); + + if (shouldBeBlocked != window->d_func()->blockedByModalWindow) { + QEvent e(shouldBeBlocked ? QEvent::WindowBlocked : QEvent::WindowUnblocked); + + window->d_func()->blockedByModalWindow = shouldBeBlocked; + QGuiApplication::sendEvent(window, &e); + } +} + +void QGuiApplicationPrivate::showModalWindow(QWindow *modal) +{ + self->modalWindowList.prepend(modal); QEvent e(QEvent::WindowBlocked); QWindowList windows = QGuiApplication::topLevelWindows(); for (int i = 0; i < windows.count(); ++i) { QWindow *window = windows.at(i); - if (!window->d_func()->blockedByModalWindow && window->windowType() != Qt::Tool && self->isWindowBlocked(window)) { - window->d_func()->blockedByModalWindow = true; - QGuiApplication::sendEvent(window, &e); - } + if (!window->d_func()->blockedByModalWindow) + updateBlockedStatus(window); } } @@ -425,10 +437,8 @@ void QGuiApplicationPrivate::hideModalWindow(QWindow *window) QWindowList windows = QGuiApplication::topLevelWindows(); for (int i = 0; i < windows.count(); ++i) { QWindow *window = windows.at(i); - if (window->d_func()->blockedByModalWindow && window->windowType() != Qt::Tool && !self->isWindowBlocked(window)) { - window->d_func()->blockedByModalWindow = false; - QGuiApplication::sendEvent(window, &e); - } + if (window->d_func()->blockedByModalWindow) + updateBlockedStatus(window); } } -- cgit v1.2.3