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

Distribute IVAN.dmg for MacOS and Fix icon for IVAN.app #525

Merged
merged 5 commits into from Feb 18, 2019
Merged

Distribute IVAN.dmg for MacOS and Fix icon for IVAN.app #525

merged 5 commits into from Feb 18, 2019

Conversation

jakwings
Copy link
Member

No description provided.

@jakwings
Copy link
Member Author

Wait a minute...

@jakwings
Copy link
Member Author

Done. Download link (will expire someday) https://github.com/jakwings/ivan/releases/download/vTest/IVAN-vTest-osx.dmg

@andrewtweber
Copy link
Member

andrewtweber commented Jan 18, 2019

Oooh nice. I'm glad we can set the icon on it. I'm seeing a white background though, is there any way to get a transparent background? Worst case scenario then red would be preferred over white.

Also pretty fuzzy. I can provide a higher resolution graphic if you tell me what size and format it should be.

screen shot 2019-01-18 at 5 54 44 pm

Which icon would everybody prefer, the pixelated one or the more realistic one?

pickaxe

@jakwings
Copy link
Member Author

transparent bg

like this?

@jakwings
Copy link
Member Author

PNG format is okay. Are you suggesting a new icon for all platforms? The sizes needed for making an icon.icns file on mac are 16, 32, 64, 128, 256, 512, 1024. The realistic one is nice.

@AlexMooney
Copy link
Contributor

When this is merged, can we get a new point release?

@ryfactor
Copy link
Member

When this is merged, can we get a new point release?

Yep sure thing :)

@andrewtweber
Copy link
Member

Without shadow

1024x1024-noshadow
512x512-noshadow
256x256-noshadow
128x128-noshadow
64x64-noshadow
32x32-noshadow
16x16-noshadow

@andrewtweber
Copy link
Member

andrewtweber commented Jan 29, 2019

As much as I'd like the realistic one, I don't think the quality will look good for > 128px or < 64px so we should probably stick with the classic. I'll leave it up to your judgement if it looks better with or without shadow

@andrewtweber
Copy link
Member

In hindsight it makes no sense for the shadow itself to be pixelated. So here's an updated version with the shadow.

16x16-shadow
32x32-shadow
64x64-shadow
128x128-shadow
256x256-shadow
512x512-shadow
1024x1024-shadow

@jakwings
Copy link
Member Author

Could the shadow be lighter?

compare1

compare2

compare3

compare4

Although the banana looks smaller now and has a glow/blur effect, it is still too large when compared to the pickaxe. I prefer the completely solid one.

Then I tried to enlarge the realistic one to 256x256 (128x128@2x) with https://waifu2x.booru.pics/
It looks pretty good even when the Dock size is set to the largest.

The icns has been pushed to ci/osx/IVAN.app/Contents/Resources/realistic.icns
512x512 and 1024x1024 are not included since I'm not sure when they would be used.

The latest dmg: (icon is not dynamically set to the Icon.bmp when run)
https://github.com/jakwings/ivan/releases/download/vTest2/IVAN-vTest2-osx.dmg

@jakwings
Copy link
Member Author

and the one using the realistic icon: https://github.com/jakwings/ivan/releases/download/vTest3/IVAN-vTest3-osx.dmg

@andrewtweber
Copy link
Member

andrewtweber commented Feb 7, 2019

After seeing both of them in action I think I prefer the pixelated one actually.

I do think it needs a bit of a shadow, I'll get you a copy of the icon with less shadow

I'll post a "poll" on the forum after the pixelated version is finalized

@andrewtweber
Copy link
Member

How is this?

16x16
32x32
64x64
128x128
256x256
512x512
1024x1024

@jakwings
Copy link
Member Author

Yeah, I have no problem with the one with less shadow.

+1 for both the pixelated & the realistic.

now I know that 512x512 (256x256@2x) is for the Finder pop-up window for preview

@jakwings
Copy link
Member Author

Two demos for trying out the icons: https://ipfs.io/ipfs/QmQtxPa5PGQE1FZCQMWrTQLFF6RWmopvjNeUFGPSAvC1Rw

Demo.app/
└── Contents/
    ├── Info.plist
    ├── MacOS/
    │   └── play
    ├── PkgInfo
    └── Resources/
        └── ivan.icns

@andrewtweber
Copy link
Member

@ryfactor ryfactor mentioned this pull request Feb 14, 2019
@andrewtweber
Copy link
Member

@iology it's overwhelmingly in favor of keeping the pixelated icon (only 5 votes but.. that's about all we can hope for). Also sounds like the modern one may have been specifically made for the website. Let's go with pixelated.

J.W added 5 commits February 16, 2019 05:08
The icon was not shown because there was a directory "ivan" with the
same basename as the icon file "ivan.icns". Set the extension .icns
explicitly to resolve the issue.
Use a pixelated icon with shadow and transparent background.
@jakwings
Copy link
Member Author

Rebased and cleaned up the commit messages.

So this icon is for all platforms. Note that this is a PR for mac only.

@andrewtweber
Copy link
Member

Looks good to merge but I'll let @fejoa take a look first as well as see what needs to be done for Windows icon to match with the slight shadow

@ryfactor
Copy link
Member

@andrewtweber excellent, I'll look into it. What are your settings for the drop shadow?

@andrewtweber
Copy link
Member

andrewtweber commented Feb 17, 2019

Oof I don't remember.

What format does Windows use, ICO?

Any way to just combine the PNGs above into layers? Or does it need a size that we don't have

@ryfactor
Copy link
Member

ryfactor commented Feb 18, 2019

Yep Windows just uses the ICO format. I just save your 256x256 png and open it in GIMP. Using GIMP I can create a variety of sizes from the largest windows size (256x256). For some reason the drop shadow goes away, but I can set it up again in GIMP and it will be preserved in the export to the .ico format.

EDIT: Almost got something working, just need the info about the drop shadow. Preferable an x-offset, y-offset, blur radius and opacity, all measured in pixels... 😨

@andrewtweber
Copy link
Member

Hmm, you should be able to just paste the individual files above as layers. Does this work?

https://drive.google.com/file/d/1HFo7XGudu5LUeS4vzOk2jF31oA8XPBh4/view?usp=sharing

@ryfactor
Copy link
Member

Thanks Andrew, I'll download those tomorrow and make the ICO.

@andrewtweber
Copy link
Member

andrewtweber commented Feb 18, 2019

The google drive file I linked to in the previous comment is an ICO :) Has all sizes from 16x16 to 256x256. Should’ve just made it for you in the first place

@ryfactor
Copy link
Member

ryfactor commented Feb 18, 2019

Excellent, thanks Andrew. I did open it for a look and found it didn't have the 48x48 pixel size needed for windows, so I added that in. It seems like the drop shadow has been preserved throughout the processing.

I added the ICO file in #533. In a hideous twist, it seems like Windows caches the old icon, so existing players might not see the updated icon if they add a shortcut to IVAN for example to their desktop folder. If they access IVAN from its own folder, then I think it will simply display our new icon :)

EDIT: Since the new ICO has the same name as the old, no further changes are needed to Ivan.rc. The build system will therefore take care of everything.

@andrewtweber andrewtweber merged commit 93efe67 into Attnam:master Feb 18, 2019
@jakwings jakwings mentioned this pull request Feb 6, 2020
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.

4 participants