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

Support patching icons for iOS 11 / Xcode 9 #12

Open
richardszalay opened this issue Nov 8, 2017 · 3 comments
Open

Support patching icons for iOS 11 / Xcode 9 #12

richardszalay opened this issue Nov 8, 2017 · 3 comments

Comments

@richardszalay
Copy link
Owner

richardszalay commented Nov 8, 2017

As raised by @sommestad of @Springworks, Xcode 9 now builds icons into the asset catalog and refers to them by name as CFBundleIconName.

Act should be updated to support this, but so far it's not entirely clear how this should happen. It looks like the icon needs to be part of the asset catalog, and also be rendered in a number of sizes/densities. Rather than looking to patch the .car asset catalog (while projects exist that do so, it's involved, and the format is subject to change), it might be best to look at whether the asset catalog used for the icon and switch it to a newly compiled asset catalog containing only the icon (and possibly background image?).

@ghost
Copy link

ghost commented Dec 11, 2017

Any news on this issue? Thanks

@richardszalay
Copy link
Owner Author

Hi @DuboisJoris, take a look through this thread for an idea of what needs to be investigated nest.

I'm not working in the iOS space at the moment (and thus have no access to a Mac) so I'm not able to investigate further myself. Please let me know if you do manage to take a look at it yourself.

@richardszalay
Copy link
Owner Author

richardszalay commented Jan 1, 2019

For anyone watching this thread, I've done some additional research and the docs seem to imply that it's totally valid to still use CFBundleIconFiles.

So as long as you're iconset contains all the up-to-date icon sizes (note the 1024x1024 app store version) then it should work.

Unless the documentation is wrong, the error you receive ("A value for the Info.plist key CFBundleIconName is missing in the bundle '<project_id>") is probably just a bad error message, and what it means is that "there are required icon sizes missing".

If someone (cc @sommestad @thatjuan) can confirm this, I can update the README and even look at some automatic validation of iconsets to catch missing required sizes early.

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

No branches or pull requests

1 participant