-
Notifications
You must be signed in to change notification settings - Fork 17.4k
Update Mac icon to be flatter, to work with Yosemite styling #3089
Conversation
It's atoms all the way down, man. |
I suggest we have @cameronmcefee or @jeejkang do this work. |
Oh whoops, that was an artifact, I should have caught that. I will update with the correct stuff later. Thanks for finding that @kevinsawicki |
This looks nice. The glassy texture of the current icon definitely looks out of place on Yosemite. This article covers the move away from glass & onto a more flat plane nicely. |
Hi @ParkerK, thanks for your contribution! Because your pull request modifies a trademarked design, I need to check with our legal department to make sure the law would allow such a modification to be made (trademark law is tricky about these thigns). Once I know if it's ok, I'll provide some feedback about how we can move forward with things. |
I wouldn't mind this for Windows either. Win8 design is more flat than Win7, so it would certainly fit in while not particularly hurting other versions. |
Just throwing a general 👍 and that I would love to see this new icon on linux as well. |
Thanks @cameronmcefee ! Understood, I'm more than happy to help to make it work with whatever guidelines they give. @batjko good idea, something like this maybe? @mehcode thanks! |
@ParkerK I don't use the metro interface, but I'm guess that would work. |
I think the Atom icon shouldn't be changed to often. Of course it would be a nice change for yosemite, but whats with the Windows 7 users ect.? If you want to change it, i would recommend one of this two, that are really nice. https://dribbble.com/shots/1615345-Atom-Editor-Yosemite-App-Icon-wo-shadow Another thread about Atom Yosemite icon: https://discuss.atom.io/t/alternative-atom-yosemite-icon/10093 |
@philipgiuliani I don't see why any of those two or three suggested icons wouldn't work for Windows. |
Ahh, yeah @batjko I went right to metro for some reason. Yes, the icon should work with both Linux and Windows, it's much flatter now, but flat isn't OS X only, and neither Linux nor Windows are unified like OS X is |
@batjko yeah you're right, now i looked over my Windows Software i use, and they all are really different... Just OS X is a little bit "strict" here :) |
Hi Everyone, We've talked it out and have decided that we won't be accepting brand-related pull requests because of the murky nature of authorship. Changing your own personal icon is totally legit as far as the license goes so have at it if you so choose, but we're going to keep the official brand updates to GitHub staff only. Sorry for the less than ideal result, but we super appreciate that you care enough about Atom to contribute. |
Fair enough. |
@cameronmcefee could you update the |
What if I just agree to transfer all ownership to you guys? Thanks for looking into this, either way |
http://appicontemplate.com/osx has a pretty relaxed license though |
As far as I know they're more concerned about artistic copyright than the tools used, in this case because @ParkerK has artistic copyright over his creation even if it does use Github's Atom icon design. It wouldn't be a massively difficult thing to get around, Github and X can draw up a contract where the work done belongs to Github entirely and irrevocably and in return the artist is paid $0.01 or something trivial along those lines, but I can understand why Github wants to avoid dealing with the slaughterhouse of legislation and regulation on this particular issue. |
I actually meant that someone from the team could just use the template and come up with an icon without having the whole contract deal. Should be easy enough. |
@kevinsawicki @benogle @nathansobo What do you guys think ? I use this personally: Classy Atom |
This icon looks even much better when used and rendered as an icon. |
Yosemite has a much flatter look to it, and all of the default Mac apps have had their icons changed. I updated the Atom icon to fit in with these theme a little better. Any feedback would be appreciated :)
Old:![atom](https://cloud.githubusercontent.com/assets/464971/3705051/b566c906-141a-11e4-96ea-44397c3e610f.png)
![f2b4523d650ae78d-64x64](https://cloud.githubusercontent.com/assets/464971/3705035/8215410e-141a-11e4-91bb-2c7ffa7c8982.png)
New: