Skip to content

Allow opening statepoint files when one is already open#177

Open
pshriwise wants to merge 1 commit intoopenmc-dev:developfrom
pshriwise:dynamic-sp-files
Open

Allow opening statepoint files when one is already open#177
pshriwise wants to merge 1 commit intoopenmc-dev:developfrom
pshriwise:dynamic-sp-files

Conversation

@pshriwise
Copy link
Collaborator

@pshriwise pshriwise commented Mar 8, 2026

Another QOL upgrade here. It's been bothering me for a while that you can't more cleanly open a new statepoint file with one already open. Feels clunky for it to fail and force the user to close the existing file manually. This takes care of that by providing a notice if one is open and optionally opening the file open dialog anyway after confirmation.

@pshriwise
Copy link
Collaborator Author

The test code is fully codex-generated. It's a lot of line changes and while I'd like to have more testing here it's not the type of testing I'm primarily concerned with. Happy to remove it if we think that's best.

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.

1 participant