Skip to content
This repository has been archived by the owner on Aug 20, 2019. It is now read-only.

Added icons to application and notifications #44

Merged
merged 2 commits into from
Sep 3, 2016

Conversation

adamfitzgibbon
Copy link
Contributor

Added a folder with the icon files. Application icon and some notifications were updated to use the new icon.

@adamfitzgibbon adamfitzgibbon mentioned this pull request Sep 1, 2016
@djw8605
Copy link
Collaborator

djw8605 commented Sep 2, 2016

I asked the designer for the source photoshop files so we could include those in the source as well. I'll attach it here when I get them.

But, I tested this change, and it all looks good. Seems to work on my Mac.

Is there anywhere else we want to include this Icon? Maybe in the "brand" of the navigation bar (top left?)

@djw8605
Copy link
Collaborator

djw8605 commented Sep 2, 2016

Add the uncompressed Adobe Illustrator file attached here. Just in case we need to update it some time in the future.

1608 HCCGo Desktop Icon_FINAL.ai.zip

@djw8605 djw8605 added this to the v0.1 Beta milestone Sep 2, 2016
@adamfitzgibbon
Copy link
Contributor Author

Yeah I can try and put that in the navbar, I think that'd be a good place to stick it.

@adamfitzgibbon
Copy link
Contributor Author

adamfitzgibbon commented Sep 2, 2016

Okay I added the icon to the navbar in place of where HCCGo was. I figured that it wasn't needed since it also displays the name at the top of the window. Also fixed some things on the navbar so that on smaller windows it correctly turns into a collapsible. And finally a new folder was added to the root directory to hold the source file for the icon and any other open source resources we might want to include in the future.

@djw8605
Copy link
Collaborator

djw8605 commented Sep 3, 2016

All looks good to me!

Fixes #43

@djw8605 djw8605 merged commit 3435461 into unlhcc:master Sep 3, 2016
@adamfitzgibbon adamfitzgibbon deleted the Icons branch October 20, 2016 16:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants