Skip to content

wxGUI: Skip size event in MapPanelBase.OnSize to let AUI lay out panes - #7941

Merged
petrasovaa merged 1 commit into
OSGeo:mainfrom
petrasovaa:wxgui-aui-onsize-skip
Sep 23, 2026
Merged

petrasovaa merged 1 commit into
OSGeo:mainfrom
petrasovaa:wxgui-aui-onsize-skip

Conversation

@petrasovaa

Copy link
Copy Markdown
Contributor

Problem with newer wxPython (4.3.1):
image

wxWidgets 3.3 (wxPython 4.3) changed wxAuiManager to bind its size handler to the managed window instead of pushing itself as an event handler. Bound handlers run newest first and stop at the first one that does not skip the event, so the map panel's own size handler, bound after the manager was created, swallowed the event. The manager never laid out its panes and the map window kept its initial size.

Diagnosed with the help of Claude Code.

wxWidgets 3.3 (wxPython 4.3) changed wxAuiManager to bind its size
handler to the managed window instead of pushing itself as an event
handler. Bound handlers run newest first and stop at the first one
that does not skip the event, so the map panel's own size handler,
bound after the manager was created, swallowed the event. The manager
never laid out its panes and the map window kept its initial size.

Diagnosed with the help of Claude Code.
@github-actions github-actions Bot added GUI wxGUI related Python Related code is in Python labels Sep 23, 2026
@petrasovaa petrasovaa added the backport to 8.5 PR needs to be backported to release branch 8.5 label Sep 23, 2026
@petrasovaa
petrasovaa enabled auto-merge (squash) September 23, 2026 14:33
@petrasovaa
petrasovaa merged commit 527588b into OSGeo:main Sep 23, 2026
26 checks passed
@github-actions github-actions Bot added this to the 8.6.0 milestone Sep 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport to 8.5 PR needs to be backported to release branch 8.5 GUI wxGUI related Python Related code is in Python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants