Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix 5676ce7a, where returning to the Welcome screen and opening a project again could break panels in the main window. #157

Merged
merged 1 commit into from
Jun 15, 2024

Conversation

DaleStan
Copy link
Collaborator

@DaleStan DaleStan commented Jun 15, 2024

To reproduce this, open the Milestones Editor, NEI Explorer, Preferences screen, a single-object selection screen (the full list of fuels or modules is most likely), the Shopping list, the module customization screen, or the module templates screen. Then select "Return to starting screen" from the main menu, load or create any project, and try to interact with any screen you previously opened.

<Escape> and maybe <Enter> will work, but mouse interaction will not work.

…ect again could break panels in the main window.
@shpaass
Copy link
Owner

shpaass commented Jun 15, 2024

Any idea why it could've happened? Were the instances disposed of?

@shpaass shpaass merged commit f22fe80 into shpaass:master Jun 15, 2024
1 check passed
@DaleStan
Copy link
Collaborator Author

Each Instance gets the first MainScreen's InputSystem, and continues to interact with that system even after the new InputSystem is created for the second MainScreen. Recreating the PseudoScreens every time forces them to reread MainScreen.Instance.InputSystem, and interact with the current system.

@DaleStan DaleStan deleted the fix-154 branch June 16, 2024 01:41
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