Added function to toggle resuming of last song when app starts #177
Conversation
|
@NNowakowski Which toggle option is this and should it be available if I have auto-updates turned on? |
This request wasn't merged yet. Once it gets added to the codebase it should be available with the next update following afterwards. |
Ah, I see. My bad, I'm fairly new to GitHub. And now I also see the "files changed" tab which explains where/how the change will be implemented. Thank you for writing up the changes and responding to my question so quickly! |
No problem. It's a feature I wanted to see aswell so seeing your request gave me the push I needed to implement this :p |
| @@ -111,7 +111,8 @@ function createMainWindow() { | |||
| win.maximize(); | |||
| } | |||
|
|
|||
| win.webContents.loadURL(config.get("url")); | |||
| const urlToLoad = config.get("resumeOnStart") ? config.get("url") : "https://music.youtube.com"; | |||
There was a problem hiding this comment.
nit: root url could be imported from default config
There was a problem hiding this comment.
Small typo: option is named resumeSongOnStart in other places and not resumeOnStart
Implemented option to toggle the resume behaviour as requested in #161