Added System Tray and Notifications #19
Conversation
-- Quit functionality working as expected -- Hide/Show functionality working perfectly (tested on windows 11) -- Now the app will attempt to open conhost directly (required for hiding and showing the console) -- Fixed crashing after the first setup -- If settings.json file is invalid it will delete it for you.
|
hey @manucabral please review this pull request. I added the feature from your TODO and added a notification that pop up when a new song is detected. I also implemented opening conhost directly to fix windows 11 Show/Hide Console issue. since windows 11 defaults to using the new windows terminal I made it kill windows terminal and start conhost directly which fixed the issue. this should also work for windows 10, but since I don't have a PC running windows 10 I'd like you to verify that yourself. there's a comment next to the kill command, move /IM cmd.exe behind /F and that should kill the cmd window the first instance opened in |
|
and don't forget to release a new release please, the current release is 4 versions behind. make sure to include icon.ico in the root folder of the new release after running the build command to avoid the file not found error |
-- It can now download icon.ico file from my website if icon.ico is not found -- Fixed minor path issue with the notifications
Added the system tray which was in the TODO, also fixed the app quitting after entering the client_id (due to using int() instead of str())
when building you must move the icon to the root directory of the build or it will throw an error when running the RPC.