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

Added Mavericks and Yosemite icons #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gingerbeardman
Copy link

Yosemite is default, edit Info.plist to change

Yosemite is default, changed Info.plist
@chanil1218
Copy link

Can you add full size icon?
32x32 size is too shoddy to see in Application directory.

@gingerbeardman
Copy link
Author

@chanil1218 Sorry, no chance of getting this from me. This icon (and app) is meant to be used as a button in the toolbar of the Finder window. You shouldn't be looking for the app in your Applications folder. Please understand.

@chanil1218
Copy link

I had to put cd to.app somewhere static directory to be referenced by toolbar.
So where I chose to put the app is Application directory.

I just wonder you had .ai file or something vector file that can be made for higher resolution icon like other applications in Application folder.

I want to thank for your effort making Yosemite icon!
I thought I will make toolbar Icon on my own. :)

      1. 오전 12:29 Matt Sephton notifications@github.com 작성:

@chanil1218 Sorry, no chance of getting this from me. This icon (and app) is meant to be used as a button in the toolbar of the Finder window. You shouldn't be looking for the app in your Applications folder. Please understand.


Reply to this email directly or view it on GitHub.

@gingerbeardman
Copy link
Author

I do not have a vector file, sorry.

@rrotter
Copy link

rrotter commented Apr 2, 2015

This looks excellent in Yosemite. Best looking icon yet since Apple broke greying out on background Finder windows (circa 10.7?).

@jbtule jbtule added the icons label Nov 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants