Skip to content

Restore the main menu when leaving the postgame host screen - #374

Merged
bnunu merged 1 commit into
OpenCommunityEdition:mainfrom
chestahh:opence-multiplayer-menu-fix
Oct 16, 2026
Merged

bnunu merged 1 commit into
OpenCommunityEdition:mainfrom
chestahh:opence-multiplayer-menu-fix

Conversation

@chestahh

Copy link
Copy Markdown

After a hosted multiplayer match or co-op level, leaving the lobby could show only the Halo background, requiring a restart.

Add the existing main_menu_if_no_history flag to the postgame host screen in the menu generator and generated XML so leaving it restores the main menu.

The postgame host map picker opens without menu history. Give it the
existing main_menu_if_no_history flag used by the lobby so leaving the
last screen restores the main menu instead of leaving only the background.
Update the generator and generated XML together. Preserve OpenCE's
existing handlers; no MCC code or regression-test files are included.
This local menu change does not require a network version bump.

Resolved the fork cherry-pick against OpenCE main 8102e71 by adding only
the flag override. Windows portable release build passed. Existing menu
teardown, touch and web checks returned 28 passes and 6 failures: missing
m.lib/pthread.h and Windows path separators in POSIX-oriented tests.
Android and web build targets were unavailable locally. Checked generator
round-trip stability and git diff --check. The two simple wrappers were
regenerated through ce_menus.widget_xml from their exported XML fields;
the original PC tag inputs were unavailable. No upstream runtime game
session or Linux build was run. Earlier scripted runtime verification
was performed on the GulchCE implementation, not this upstream revision.

(cherry picked from commit c811c7f)
@bnunu
bnunu self-requested a review October 16, 2026 04:13
@bnunu
bnunu merged commit f90d955 into OpenCommunityEdition:main Oct 16, 2026
@chestahh
chestahh deleted the opence-multiplayer-menu-fix branch October 16, 2026 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants