From 870f19f44b1eb2b93c7151dd48b8d9cd471acae8 Mon Sep 17 00:00:00 2001 From: Friedemann Kleint Date: Fri, 27 Jan 2012 14:09:36 +0100 Subject: Add native event filters to Windows, forward to Widgets. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Use prototypically for qwizard_win.cpp. Change-Id: I075e81ae1bc3d62d9f27e51c73c800ffd71cbcd6 Reviewed-by: Samuel Rødal --- src/widgets/dialogs/qwizard.cpp | 13 ++-- src/widgets/dialogs/qwizard.h | 4 +- src/widgets/dialogs/qwizard_win.cpp | 17 ++--- src/widgets/graphicsview/qgraphicswidget.h | 4 -- src/widgets/kernel/qwidget.cpp | 105 +++++------------------------ src/widgets/kernel/qwidget.h | 14 +--- src/widgets/kernel/qwidgetwindow_qpa.cpp | 5 ++ src/widgets/kernel/qwidgetwindow_qpa_p.h | 1 + 8 files changed, 40 insertions(+), 123 deletions(-) (limited to 'src/widgets') diff --git a/src/widgets/dialogs/qwizard.cpp b/src/widgets/dialogs/qwizard.cpp index 2c56ffc391..4e707ab7c1 100644 --- a/src/widgets/dialogs/qwizard.cpp +++ b/src/widgets/dialogs/qwizard.cpp @@ -3205,16 +3205,17 @@ void QWizard::paintEvent(QPaintEvent * event) #endif } -#if defined(Q_WS_WIN) +#if defined(Q_OS_WIN) /*! \reimp */ -bool QWizard::winEvent(MSG *message, long *result) +bool QWizard::nativeEvent(const QByteArray &eventType, void *message, long *result) { #if !defined(QT_NO_STYLE_WINDOWSVISTA) Q_D(QWizard); - if (d->isVistaThemeEnabled()) { - const bool winEventResult = d->vistaHelper->handleWinEvent(message, result); + if (d->isVistaThemeEnabled() && eventType == QByteArrayLiteral("windows_generic_MSG")) { + MSG *windowsMessage = static_cast(message); + const bool winEventResult = d->vistaHelper->handleWinEvent(windowsMessage, result); if (QVistaHelper::vistaState() != d->vistaState) { d->vistaState = QVistaHelper::vistaState(); d->vistaStateChanged = true; @@ -3222,10 +3223,10 @@ bool QWizard::winEvent(MSG *message, long *result) } return winEventResult; } else { - return QDialog::winEvent(message, result); + return QDialog::nativeEvent(eventType, message, result); } #else - return QDialog::winEvent(message, result); + return QDialog::nativeEvent(eventType, message, result); #endif } #endif diff --git a/src/widgets/dialogs/qwizard.h b/src/widgets/dialogs/qwizard.h index cd8920671f..caa9d5e648 100644 --- a/src/widgets/dialogs/qwizard.h +++ b/src/widgets/dialogs/qwizard.h @@ -189,8 +189,8 @@ protected: bool event(QEvent *event); void resizeEvent(QResizeEvent *event); void paintEvent(QPaintEvent *event); -#if defined(Q_WS_WIN) - bool winEvent(MSG * message, long * result); +#ifdef Q_OS_WIN + bool nativeEvent(const QByteArray &eventType, void * message, long * result); #endif void done(int result); virtual void initializePage(int id); diff --git a/src/widgets/dialogs/qwizard_win.cpp b/src/widgets/dialogs/qwizard_win.cpp index 3efd8274bd..d60a47f2c2 100644 --- a/src/widgets/dialogs/qwizard_win.cpp +++ b/src/widgets/dialogs/qwizard_win.cpp @@ -399,12 +399,13 @@ bool QVistaHelper::winEvent(MSG* msg, long* result) *result = DefWindowProc(msg->hwnd, msg->message, msg->wParam, msg->lParam); break; } - case WM_NCCALCSIZE: { - NCCALCSIZE_PARAMS* lpncsp = (NCCALCSIZE_PARAMS*)msg->lParam; - *result = DefWindowProc(msg->hwnd, msg->message, msg->wParam, msg->lParam); - lpncsp->rgrc[0].top -= (vistaState() == VistaAero ? titleBarSize() : 0); - break; - } +// case WM_NCCALCSIZE: { #fixme: If the frame size is changed, it needs to be communicated to the QWindow. +// NCCALCSIZE_PARAMS* lpncsp = (NCCALCSIZE_PARAMS*)msg->lParam; +// *result = DefWindowProc(msg->hwnd, msg->message, msg->wParam, msg->lParam); +// lpncsp->rgrc[0].top -= (vistaState() == VistaAero ? titleBarSize() : 0); +// +// break; +// } default: retval = false; } @@ -480,8 +481,8 @@ bool QVistaHelper::handleWinEvent(MSG *message, long *result) if (wizard->wizardStyle() == QWizard::AeroStyle && vistaState() == VistaAero) { status = winEvent(message, result); if (message->message == WM_NCCALCSIZE) { - if (status) - collapseTopFrameStrut(); +// if (status) #fixme +// collapseTopFrameStrut(); } else if (message->message == WM_NCPAINT) { wizard->update(); } diff --git a/src/widgets/graphicsview/qgraphicswidget.h b/src/widgets/graphicsview/qgraphicswidget.h index c073de32ef..e0bb591231 100644 --- a/src/widgets/graphicsview/qgraphicswidget.h +++ b/src/widgets/graphicsview/qgraphicswidget.h @@ -211,17 +211,13 @@ protected: virtual bool focusNextPrevChild(bool next); void focusOutEvent(QFocusEvent *event); virtual void hideEvent(QHideEvent *event); - //virtual bool macEvent(EventHandlerCallRef caller, EventRef event); //virtual int metric(PaintDeviceMetric m ) const; virtual void moveEvent(QGraphicsSceneMoveEvent *event); virtual void polishEvent(); - //virtual bool qwsEvent(QWSEvent *event); //void resetInputContext (); virtual void resizeEvent(QGraphicsSceneResizeEvent *event); virtual void showEvent(QShowEvent *event); //virtual void tabletEvent(QTabletEvent *event); - //virtual bool winEvent(MSG *message, long *result); - //virtual bool x11Event(XEvent *event); virtual void hoverMoveEvent(QGraphicsSceneHoverEvent *event); virtual void hoverLeaveEvent(QGraphicsSceneHoverEvent *event); virtual void grabMouseEvent(QEvent *event); diff --git a/src/widgets/kernel/qwidget.cpp b/src/widgets/kernel/qwidget.cpp index af3941c231..4402b497ad 100644 --- a/src/widgets/kernel/qwidget.cpp +++ b/src/widgets/kernel/qwidget.cpp @@ -8963,111 +8963,36 @@ void QWidget::hideEvent(QHideEvent *) { } -/* - \fn QWidget::x11Event(MSG *) - - This special event handler can be reimplemented in a subclass to receive - native X11 events. - - In your reimplementation of this function, if you want to stop Qt from - handling the event, return true. If you return false, this native event - is passed back to Qt, which translates it into a Qt event and sends it to - the widget. - - \note Events are only delivered to this event handler if the widget is - native. - - \warning This function is not portable. - - \sa QApplication::x11EventFilter(), QWidget::winId() -*/ - - -#if defined(Q_WS_MAC) - /*! - \fn bool QWidget::macEvent(EventHandlerCallRef caller, EventRef event) - This special event handler can be reimplemented in a subclass to - receive native Macintosh events. - - The parameters are a bit different depending if Qt is build against Carbon - or Cocoa. In Carbon, \a caller and \a event are the corresponding - EventHandlerCallRef and EventRef that correspond to the Carbon event - handlers that are installed. In Cocoa, \a caller is always 0 and the - EventRef is the EventRef generated from the NSEvent. + receive native platform events identified by \a eventType + which are passed in the \a message parameter. In your reimplementation of this function, if you want to stop the - event being handled by Qt, return true. If you return false, this - native event is passed back to Qt, which translates the event into - a Qt event and sends it to the widget. + event being handled by Qt, return true and set \a result. + If you return false, this native event is passed back to Qt, + which translates the event into a Qt event and sends it to the widget. - \warning This function is not portable. + \note Events are only delivered to this event handler if the widget is + has a native Window handle. - \warning This function was not called inside of Qt until Qt 4.4. - If you need compatibility with earlier versions of Qt, consider QApplication::macEventFilter() instead. + \note This function superseedes the event filter functions + x11Event(), winEvent() and macEvent() of Qt 4. - \sa QApplication::macEventFilter() + \table + \header \i Platform \i Event Type Identifier \i Message Type \i Result Type + \row \i Windows \i "windows_generic_MSG" \i MSG * \i LRESULT + \endtable */ -bool QWidget::macEvent(EventHandlerCallRef, EventRef) -{ - return false; -} - -#endif -#if defined(Q_WS_WIN) - -/*! - This special event handler can be reimplemented in a subclass to - receive native Windows events which are passed in the \a message - parameter. - - In your reimplementation of this function, if you want to stop the - event being handled by Qt, return true and set \a result to the value - that the window procedure should return. If you return false, this - native event is passed back to Qt, which translates the event into - a Qt event and sends it to the widget. - - \warning This function is not portable. - - \sa QApplication::winEventFilter() -*/ -bool QWidget::winEvent(MSG *message, long *result) +bool QWidget::nativeEvent(const QByteArray &eventType, void *message, long *result) { + Q_UNUSED(eventType); Q_UNUSED(message); Q_UNUSED(result); return false; } -#endif -#if defined(Q_WS_X11) - -/*! - \fn bool QWidget::x11Event(XEvent *event) - - This special event handler can be reimplemented in a subclass to receive - native X11 events passed in the \a event parameter. - - In your reimplementation of this function, if you want to stop Qt from - handling the event, return true. If you return false, this native event - is passed back to Qt, which translates it into a Qt event and sends it to - the widget. - - \note Events are only delivered to this event handler if the widget is - native. - - \warning This function is not portable. - - \sa QApplication::x11EventFilter(), QWidget::winId() -*/ -bool QWidget::x11Event(XEvent *) -{ - return false; -} - -#endif - /*! Ensures that the widget has been polished by QStyle (i.e., has a proper font and palette). diff --git a/src/widgets/kernel/qwidget.h b/src/widgets/kernel/qwidget.h index f21b2118a4..f24941d856 100644 --- a/src/widgets/kernel/qwidget.h +++ b/src/widgets/kernel/qwidget.h @@ -660,19 +660,7 @@ protected: virtual void showEvent(QShowEvent *); virtual void hideEvent(QHideEvent *); - -#if defined(Q_WS_MAC) - virtual bool macEvent(EventHandlerCallRef, EventRef); -#endif -#if defined(Q_WS_WIN) - virtual bool winEvent(MSG *message, long *result); -#endif -#if defined(Q_WS_X11) - virtual bool x11Event(XEvent *); -#endif -#if defined(Q_WS_QWS) - virtual bool qwsEvent(QWSEvent *); -#endif + virtual bool nativeEvent(const QByteArray &eventType, void *message, long *result); // Misc. protected functions virtual void changeEvent(QEvent *); diff --git a/src/widgets/kernel/qwidgetwindow_qpa.cpp b/src/widgets/kernel/qwidgetwindow_qpa.cpp index e546df755d..e7ba4853a0 100644 --- a/src/widgets/kernel/qwidgetwindow_qpa.cpp +++ b/src/widgets/kernel/qwidgetwindow_qpa.cpp @@ -431,4 +431,9 @@ void QWidgetWindow::handleWindowStateChangedEvent(QWindowStateChangeEvent *event QGuiApplication::sendSpontaneousEvent(m_widget, &widgetEvent); } +bool QWidgetWindow::nativeEvent(const QByteArray &eventType, void *message, long *result) +{ + return m_widget->nativeEvent(eventType, message, result); +} + QT_END_NAMESPACE diff --git a/src/widgets/kernel/qwidgetwindow_qpa_p.h b/src/widgets/kernel/qwidgetwindow_qpa_p.h index d7fdb5d4de..1e8813fddc 100644 --- a/src/widgets/kernel/qwidgetwindow_qpa_p.h +++ b/src/widgets/kernel/qwidgetwindow_qpa_p.h @@ -79,6 +79,7 @@ protected: void handleDragEvent(QEvent *); void handleExposeEvent(QExposeEvent *); void handleWindowStateChangedEvent(QWindowStateChangeEvent *event); + bool nativeEvent(const QByteArray &eventType, void *message, long *result); private: void updateGeometry(); -- cgit v1.2.3