Skip to content
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

issue #12: implemented basic download management #16

Merged
merged 9 commits into from
May 1, 2015

Conversation

art-divin
Copy link
Contributor

No description provided.

@art-divin
Copy link
Contributor Author

Implemented listing of active downloads

@insidegui
Copy link
Owner

I will make some modifications and merge this as soon as I can. Thank you.

@art-divin
Copy link
Contributor Author

@insidegui please tell me what would you like to change and I'll work on it 😉

@insidegui
Copy link
Owner

I didn't revise the code yet, but to be honest I didn't like the UI at all.

We need to get rid of the table headers and show the session's title instead of the raw URL. I think Safari's download manager would be a good reference.

@art-divin
Copy link
Contributor Author

@insidegui I've made improvements, please check it out and send some feedback when you'll have a chance, thanks!

@insidegui
Copy link
Owner

Good. I am making some changes to the UI right now, It's close.

@insidegui
Copy link
Owner

Almost done :)

captura de tela 2015-05-01 as 14 05 44

@art-divin
Copy link
Contributor Author

Looks very nice :) If its not a secret, where did you get icons? Some freeware asset resource or something?

@insidegui
Copy link
Owner

There are standard images on OS X, the "cancel" image is called NSStopProgressFreestandingTemplate and the "refresh" image (not shown in the screenshot above) is called NSRefreshFreestandingTemplate.

@insidegui insidegui merged commit c2128b9 into insidegui:master May 1, 2015
@art-divin art-divin deleted the issue-12 branch May 1, 2015 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants