-
Notifications
You must be signed in to change notification settings - Fork 47
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
Add download path option #17
base: master
Are you sure you want to change the base?
Conversation
Gentle ping @DenBeke BTW, I have never coded swift or used XCode before. There probably are many other ways to implement this feature and I may choose the worst one😂. Hope you don't feel disturbed when reviewing this. |
Signed-off-by: Jianbai Ye <[email protected]>
Don't worry, I'm also not an experienced macOS programmer :) I was thinking, wouldn't it make more sense to have a button (probably with just a cogs icon) that opens a preferences window where more settings could be grouped together in the future? That would also less clutter the main screen which I want to keep as simple as possible. |
@gusye1234 I tried your PR and it seems that if I try to run it two consecutive times, the field is not reappearing again. Could you fix that? Thanks |
My apologies for the delay in reviewing this, I have been quite busy. |
Just a quick question, where you guys would recommend someone to start learning swift? |
Long time ago I saw some live demos... So yeah, can't really help you with real resources, but Apple has quite a lot of information on their developer website: https://developer.apple.com/swift/resources/ Hope this helps... |
This pr involved slight UI modification to add a download path configure