Re: Key events in wxPopupTransientWindow broken on some platforms (#17578)
Richard Thomson
LegalizeAdulthood left a comment (wxWidgets/wxWidgets#17578)I checked the current tree and this appears to have already been addressed by 07f64c3 ("Add wxPU_CONTAINS_CONTROLS style for wxPopupWindow").
On MSW, wxPU_CONTAINS_CONTROLS makes wxPopupWindow use a focusable WS_POPUP implementation for popups containing controls. wxPopupTransientWindow::Popup() also focuses the provided descendant control when this flag is used, which is the path needed for controls such as wxTextCtrl to receive key events inside the popup.
The popup sample now exercises this path: it creates SimpleTransientPopup with wxPU_CONTAINS_CONTROLS and includes a wxTextCtrl labelled "Try to type here".
I also verified that the relevant targets build locally:
cmake --build C:\code\wxWidgets\build-rt-default --target popup --config Debug
cmake --build C:\code\wxWidgets\build-rt-default --target test_gui --config Debug
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you authored the thread.