-
Notifications
You must be signed in to change notification settings - Fork 206
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
Updated Yabause icons #347
base: master
Are you sure you want to change the base?
Conversation
Updated for all platforms - welcome to 2016.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks much better. What's the reason for the dimension changes (i.e. 32x32 to 512x512)?
I made the icon for the desktop version larger because desktop icons are usually bigger than 32x32 nowadays.. 512x512 may be too much though but something like 64x64 or 128x128 should be about right for modern OS |
512x512 isnt too much, you are just prepared to 8K resolutions. ;) - you must bring 512x512 back and be sure your icons dont have jagged borders AT ALL. ;) |
There will always be jagged borders since it's raster... anyhow the change doesn't matter because it only affected the png files, the .ico and icns still contains up to 512x512 ;) |
I see, good to know 512 still there, too bad about jagged stuff. |
English language - typos fix
Updated for all platforms - welcome to 2016 :P