Skip to content

fix(multiblock): close Multiblock menu if it is no longer valid#119

Merged
ACCBDD merged 3 commits intoACCBDD:1.20.1from
Rurla:fix_gyrofuge_menu
Feb 20, 2026
Merged

fix(multiblock): close Multiblock menu if it is no longer valid#119
ACCBDD merged 3 commits intoACCBDD:1.20.1from
Rurla:fix_gyrofuge_menu

Conversation

@Rurla
Copy link
Contributor

@Rurla Rurla commented Feb 10, 2026

In some cases, the Multiblock menu remains open if the Multiblock becomes invalid while the menu is open. This PR fixes the issue. Also fixes several potential NPEs.

@Rurla Rurla changed the title fix(gyrofuge): close Gyrofuge menu if it is no longer valid fix(gyrofuge): close Multiblock menu if it is no longer valid Feb 11, 2026
@Rurla Rurla changed the title fix(gyrofuge): close Multiblock menu if it is no longer valid fix(multiblock): close Multiblock menu if it is no longer valid Feb 11, 2026
@ACCBDD ACCBDD self-requested a review February 20, 2026 22:30
ACCBDD
ACCBDD previously requested changes Feb 20, 2026
Copy link
Owner

@ACCBDD ACCBDD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good! thank you for documenting your util method as well, i really appreciate that :)

@ACCBDD ACCBDD merged commit f549170 into ACCBDD:1.20.1 Feb 20, 2026
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