From 4cf1a9bf4af038e88d2a9fffdc7f919fa91e7572 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Sun, 11 Dec 2011 17:03:56 +0000 Subject: [PATCH] Ensure that Enter key presses are never stolen from wxButton in wxMSW. This commit fixes the following bug: when an in-place editor control containing an embedded button was used in wxDataViewCtrl, pressing Enter on the button would close the editor, accepting changes, instead as (generic) wxDataViewCtrl intercepts WXK_RETURN in its EVT_CHAR_HOOK handler. To prevent this from happening, wxButton now handles EVT_CHAR_HOOK itself and never lets the parent window intercept it if it's for WXK_RETURN. To ensure that normal wxEVT_KEY_DOWN and wxEVT_CHAR are still generated in this case, wxButton handler calls the new wxKeyEvent::DoAllowNextEvent() method that was added to allow suppressing EVT_CHAR_HOOK only, without affecting the subsequent events. DoAllowNextEvent() is currently only used in wxMSW but support for it was also added to wxGTK and (both) wxOSX ports. See #9102. git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@69984 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- include/wx/event.h | 16 ++++++++++++++ include/wx/msw/button.h | 3 +++ interface/wx/event.h | 48 +++++++++++++++++++++++++++++++++++------ src/gtk/window.cpp | 3 ++- src/msw/button.cpp | 23 ++++++++++++++++++++ src/msw/window.cpp | 7 ++++-- src/osx/carbon/app.cpp | 2 +- src/osx/cocoa/window.mm | 3 ++- 8 files changed, 93 insertions(+), 12 deletions(-) diff --git a/include/wx/event.h b/include/wx/event.h index 7b4d53c6a2..fb45b71c55 100644 --- a/include/wx/event.h +++ b/include/wx/event.h @@ -1707,6 +1707,15 @@ public: // Get Y position wxCoord GetY() const { return m_y; } + // Can be called from wxEVT_CHAR_HOOK handler to allow generation of normal + // key events even though the event had been handled (by default they would + // not be generated in this case). + void DoAllowNextEvent() { m_allowNext = true; } + + // Return the value of the "allow next" flag, for internal use only. + bool IsNextEventAllowed() const { return m_allowNext; } + + virtual wxEvent *Clone() const { return new wxKeyEvent(*this); } virtual wxEventCategory GetEventCategory() const { return wxEVT_CATEGORY_USER_INPUT; } @@ -1750,6 +1759,8 @@ private: { if ( m_eventType == wxEVT_CHAR_HOOK ) m_propagationLevel = wxEVENT_PROPAGATE_MAX; + + m_allowNext = false; } // Copy only the event data present in this class, this is used by @@ -1768,6 +1779,11 @@ private: #endif } + // If this flag is true, the normal key events should still be generated + // even if wxEVT_CHAR_HOOK had been handled. By default it is false as + // handling wxEVT_CHAR_HOOK suppresses all the subsequent events. + bool m_allowNext; + DECLARE_DYNAMIC_CLASS(wxKeyEvent) }; diff --git a/include/wx/msw/button.h b/include/wx/msw/button.h index ebf260e87f..d1f4c4e364 100644 --- a/include/wx/msw/button.h +++ b/include/wx/msw/button.h @@ -77,6 +77,9 @@ private: m_authNeeded = false; } + void OnCharHook(wxKeyEvent& event); + + wxDECLARE_EVENT_TABLE(); wxDECLARE_DYNAMIC_CLASS_NO_COPY(wxButton); }; diff --git a/interface/wx/event.h b/interface/wx/event.h index 31b9606d71..5cb9c497aa 100644 --- a/interface/wx/event.h +++ b/interface/wx/event.h @@ -1373,13 +1373,19 @@ enum wxKeyCategoryFlags events and so gives the parent window an opportunity to modify the keyboard handling of its children, e.g. it is used internally by wxWidgets in some ports to intercept pressing Esc key in any child of a - dialog to close the dialog itself when it's pressed. If the event is - handled, i.e. the handler doesn't call wxEvent::Skip(), neither @c - wxEVT_KEY_DOWN nor @c wxEVT_CHAR events will be generated (although @c - wxEVT_KEY_UP still will be). Notice that this event is not generated - when the mouse is captured as it is considered that the window which - has the capture should receive all the keyboard events too without - allowing its parent wxTopLevelWindow to interfere with their processing. + dialog to close the dialog itself when it's pressed. By default, if + this event is handled, i.e. the handler doesn't call wxEvent::Skip(), + neither @c wxEVT_KEY_DOWN nor @c wxEVT_CHAR events will be generated + (although @c wxEVT_KEY_UP still will be), i.e. it replaces the normal + key events. However by calling the special DoAllowNextEvent() method + you can handle @c wxEVT_CHAR_HOOK and still allow normal events + generation. This is something that is rarely useful but can be required + if you need to prevent a parent @c wxEVT_CHAR_HOOK handler from running + without suppressing the normal key events. Finally notice that this + event is not generated when the mouse is captured as it is considered + that the window which has the capture should receive all the keyboard + events too without allowing its parent wxTopLevelWindow to interfere + with their processing. @endEventTable @see wxKeyboardState @@ -1522,6 +1528,34 @@ public: Returns the Y position (in client coordinates) of the event. */ wxCoord GetY() const; + + /** + Allow normal key events generation. + + Can be called from @c wxEVT_CHAR_HOOK handler to indicate that the + generation of normal events should @em not be suppressed, as it happens + by default when this event is handled. + + The intended use of this method is to allow some window object to + prevent @c wxEVT_CHAR_HOOK handler in its parent window from running by + defining its own handler for this event. Without calling this method, + this would result in not generating @c wxEVT_KEY_DOWN nor @c wxEVT_CHAR + events at all but by calling it you can ensure that these events would + still be generated, even if @c wxEVT_CHAR_HOOK event was handled. + + @since 2.9.3 + */ + void DoAllowNextEvent(); + + /** + Returns @true if DoAllowNextEvent() had been called, @false by default. + + This method is used by wxWidgets itself to determine whether the normal + key events should be generated after @c wxEVT_CHAR_HOOK processing. + + @since 2.9.3 + */ + bool IsNextEventAllowed() const; }; diff --git a/src/gtk/window.cpp b/src/gtk/window.cpp index f7ef4a6db0..fbbbd782e3 100644 --- a/src/gtk/window.cpp +++ b/src/gtk/window.cpp @@ -830,7 +830,8 @@ bool SendCharHookEvent(const wxKeyEvent& event, wxWindow *win) if ( !g_captureWindow ) { wxKeyEvent eventCharHook(wxEVT_CHAR_HOOK, event); - if ( win->HandleWindowEvent(eventCharHook) ) + if ( win->HandleWindowEvent(eventCharHook) + && !event.IsNextEventAllowed() ) return true; } diff --git a/src/msw/button.cpp b/src/msw/button.cpp index e87917bd64..3e9bfb5739 100644 --- a/src/msw/button.cpp +++ b/src/msw/button.cpp @@ -62,6 +62,10 @@ // macros // ---------------------------------------------------------------------------- +BEGIN_EVENT_TABLE(wxButton, wxButtonBase) + EVT_CHAR_HOOK(wxButton::OnCharHook) +END_EVENT_TABLE() + // ============================================================================ // implementation // ============================================================================ @@ -370,6 +374,25 @@ void wxButton::Command(wxCommandEvent & event) // event/message handlers // ---------------------------------------------------------------------------- +void wxButton::OnCharHook(wxKeyEvent& event) +{ + // We want to ensure that the button always processes Enter key events + // itself, even if it's inside some control that normally takes over them + // (this happens when the button is part of an in-place editor control for + // example). + if ( event.GetKeyCode() == WXK_RETURN ) + { + // We should ensure that subsequent key events are still generated even + // if we did handle EVT_CHAR_HOOK (normally this would suppress their + // generation). + event.DoAllowNextEvent(); + } + else + { + event.Skip(); + } +} + bool wxButton::MSWCommand(WXUINT param, WXWORD WXUNUSED(id)) { bool processed = false; diff --git a/src/msw/window.cpp b/src/msw/window.cpp index 506bc5175a..c7bdf169d2 100644 --- a/src/msw/window.cpp +++ b/src/msw/window.cpp @@ -6644,8 +6644,11 @@ wxKeyboardHook(int nCode, WORD wParam, DWORD lParam) if ( handler && handler->ProcessEvent(event) ) { - // processed - return 1; + if ( !event.IsNextEventAllowed() ) + { + // Stop processing of this event. + return 1; + } } } } diff --git a/src/osx/carbon/app.cpp b/src/osx/carbon/app.cpp index d2378b1df9..95b1ab78b0 100644 --- a/src/osx/carbon/app.cpp +++ b/src/osx/carbon/app.cpp @@ -1650,7 +1650,7 @@ bool wxApp::MacSendCharEvent( wxWindow* focus , long keymessage , long modifiers { wxKeyEvent eventCharHook(wxEVT_CHAR_HOOK, event); handled = focus->HandleWindowEvent( eventCharHook ); - if ( handled && eventCharHook.GetSkipped() ) + if ( handled && eventCharHook.IsNextEventAllowed() ) handled = false ; } diff --git a/src/osx/cocoa/window.mm b/src/osx/cocoa/window.mm index 84f694dd02..3adaae15f4 100644 --- a/src/osx/cocoa/window.mm +++ b/src/osx/cocoa/window.mm @@ -2306,7 +2306,8 @@ bool wxWidgetCocoaImpl::DoHandleKeyEvent(NSEvent *event) if ( wxevent.GetEventType() == wxEVT_KEY_DOWN ) { wxKeyEvent eventHook(wxEVT_CHAR_HOOK, wxevent); - if ( GetWXPeer()->OSXHandleKeyEvent(eventHook) ) + if ( GetWXPeer()->OSXHandleKeyEvent(eventHook) + && !eventHook.IsNextEventAllowed() ) return true; } -- 2.45.2