nutdotnet / WinNUT-Client

WinForms-based client for monitoring your Uninterruptible Power Supply (UPS) connected to a Network UPS Tools (NUT) server.
GNU General Public License v3.0
186 stars 18 forks source link

Unable to exit when Close to tray and Minimize to tray are enabled #105

Closed masterwishx closed 9 months ago

masterwishx commented 9 months ago

cant exit from app . nothing happening. only can exit by kill app from Ctrl+Alt+del menu When press exit on icon :

image

or when press Exit in App :

image

masterwishx commented 9 months ago

i found that exit by mistake ,take function of close to tray . if i disable function close to tray in menu in settings, then exit working normal . i can exit from program.

image

masterwishx commented 9 months ago

When i disable close to systray from menu and can exit:

image

UltrashRicco commented 9 months ago

Same behaviour here, thank you for the tip consisting in NOT checking "close to tray", at least I can close Winnut if necessary, without having to kill the process.

The expected behaviour would be that when right-clicking on the tray icon and selecting "exit", Winnut should actually exit. Currently, it would appear that it keeps minimizing to the tray.

Thanks!

gbakeman commented 9 months ago

Hi all,

Another quick fix for this one. After applying it, exiting seems to work normally now. You can try a debug build here once it finishes building, and I'm planning on making this a part of the next pre-release.

UltrashRicco commented 9 months ago

@gbakeman I tested the build and it does work as expected now, thank you! :)

gbakeman commented 9 months ago

Excellent, thank you for the feedback. I'll merge this and hopefully have a release out today.

masterwishx commented 9 months ago

Thanks for fixing this and auto reconnect bugs.

Also after update I saw Logs folder moved to program folder instead in AppData/Roaming folder.

gbakeman commented 9 months ago

Thanks for fixing this and auto reconnect bugs.

Also after update I saw Logs folder moved to program folder instead in AppData/Roaming folder.

I noticed that, too. :( I think I may have broken something with the new build configuration. You can track a fix in #110

masterwishx commented 9 months ago

I noticed that, too. :( I think I may have broken something with the new build configuration

also i was looking in commits and cant find changes about logs dir ...

gbakeman commented 9 months ago

@masterwishx I wrote up an explanation in the other bug, hope that answers your question.