CaptureCoop / SnipSniper

The Screenshot Tool
https://SnipSniper.net
MIT License
3 stars 0 forks source link

ConfigWindow: After a save you are able to open a second config window #123

Closed SvenWollinger closed 2 years ago

SvenWollinger commented 2 years ago

This happens because saving restarts the tray icons, meaning the configWindow variable is null again.

SvenWollinger commented 2 years ago

Idea: Add a CacheManager where we can cache things globally Edit: Id use this to store that we have a window open, maybe even a reference to it, this could be really useful for situations like this

SvenWollinger commented 2 years ago

Better idea: Dont restart Sniper instances, only restart the popup menu. How did i not come up with that sooner... Edit: For tomorrow, make Sniper only refresh the popup menu (We still need to remember to save the new config window if it restarts though.) This will also solve the issue in the editor window if we remember the window there.

SvenWollinger commented 2 years ago

Fixed :^) Now you can only have one config window open at the same time, as having more then one can lead to out of sync issues. Lovely.