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

Allow standalone app to open multiple files, each in independent windows #123

Merged
merged 2 commits into from
Jan 4, 2021

Conversation

jpgill86
Copy link
Collaborator

@jpgill86 jpgill86 commented Dec 25, 2020

Closes #122.

StandAloneViewer was replaced by WindowManager, which handles spawning a new MainViewer each time the Open menu action is used. WindowManager also cleans up file locks and deallocates memory when a MainViewer is closed.

Additionally, when the app is run from the command line without arguments, the RawNeoOpenDialog is launched immediately, rather than the user being required to launch it manually with the Open menu action.

Finally, many of the command line interface's help and error messages have been improved.

`StandAloneViewer` was replaced by `WindowManager`, which handles spawning a new `MainViewer` each time the Open menu action is used. `WindowManager` also cleans up file locks and deallocates memory when a `MainViewer` is closed.

Additionally, when the app is run from the command line without arguments, the `RawNeoOpenDialog` is launched immediately, rather than the user being required to launch it manually with the Open menu action.
@jpgill86
Copy link
Collaborator Author

@samuelgarcia: Do you like this solution to #122?

@samuelgarcia
Copy link
Collaborator

Hi Jeffrey. Happy new year.
Sorry for long delay.
Vaccations are super long in France.
This looks good.
I would have also chossen this way i think.

@jpgill86
Copy link
Collaborator Author

jpgill86 commented Jan 4, 2021

Great, I will merge then.

France sounds nice! 😁

@jpgill86 jpgill86 merged commit bcae787 into NeuralEnsemble:master Jan 4, 2021
@jpgill86 jpgill86 deleted the standalone-multi-windows branch January 4, 2021 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Standalone app's File > Open action fails if a file was already opened
2 participants