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

feat: make ionic cordova resources comply with Android O adaptive icons #2132

Closed
eikooc opened this issue Apr 25, 2017 · 27 comments
Closed

feat: make ionic cordova resources comply with Android O adaptive icons #2132

eikooc opened this issue Apr 25, 2017 · 27 comments

Comments

@eikooc
Copy link
Contributor

eikooc commented Apr 25, 2017

Hello. This is a feature request for Android O - Adaptive icons integration with the ionic-cli.

Make the ionic resources cli command able to add the necessary information and convert an extra background file or extract an background layer from a .psd file to be able to use the new Adaptive icons in Android O.

@RafaelKr
Copy link

RafaelKr commented Oct 7, 2017

Hi, I really would love to see this feature. Is this already on the roadmap?

@captaincole
Copy link

Hey guys, just backing up these claims. Would really appreciate this so my icon looks great with the pixel and pixel 2.

screen shot 2017-11-29 at 2 15 21 pm

@imhoffd imhoffd added this to the CLI Future milestone Dec 1, 2017
@steveacalabro
Copy link

+1

5 similar comments
@ghost
Copy link

ghost commented Dec 20, 2017

+1

@nicksalt
Copy link

+1

@ElieSauveterre
Copy link

+1

@robinbonnes
Copy link

+1

@askilondz
Copy link

+1

@imhoffd imhoffd changed the title feat: make ionic resources comply with Android O adaptive icons feat: make ionic cordova resources comply with Android O adaptive icons Feb 28, 2018
@danicastanos
Copy link

+1

4 similar comments
@herrskytte
Copy link

+1

@jaclynjessup
Copy link

+1

@JamboBuenna
Copy link

+1

@gelder-ux
Copy link

+1

@eikooc
Copy link
Contributor Author

eikooc commented Mar 24, 2018

Could everyone stop writing +1? Just add a "thumbs up" reaction if you think this issue is relevant

@imhoffd
Copy link
Contributor

imhoffd commented Mar 24, 2018

Agreed--keep in mind new replies (even +1s) email all subscribers.

I'll leave my thoughts about adaptive icons for Cordova:

I understand the Ionic CLI has been falling behind in this case. Even with almost a year old issue I still haven't gotten around to adding this feature. We don't have a "roadmap", but we do use milestones, to a degree: https://github.com/ionic-team/ionic-cli/milestones

The ionic cordova resources command uploads source images to Ionic servers, converting and downloading them real-time. Up until adaptive icons, the only operations on these source images have been resize and maybe crop. There's also only been one input image. Adaptive icons require two input images, unless a solid color is used for the back layer. They also require direct edits to the XML (Cordova doesn't have this feature built-in like the other resources). It's an entirely new type of icon that caught me off guard. Couple these realities with the fact that I'm not sure what the future of ionic cordova resources is: will it become a much more reliable paid service for Ionic Pro? will there be a local version in the CLI that does the image manipulations? I will be discussing it with my colleagues soon.

We must also keep in mind the onset of Capacitor with the Ionic CLI. In Capacitor, the scope of native configuration is minuscule compared to Cordova. This wouldn't even be an issue. Android Studio has a GUI for Adaptive Icons, and that's what you'd use in Capacitor projects.

So I'm personally still unclear about where this issue will go. For now, I'll keep it in the "Future" milestone, and I'll reply back when I have more info.

@rajanshahsa
Copy link

@thielCole Can you please explain to me how you achieve adaptive icon?

Did you create a separate icon (png) file? Or some other way helps you to accomplish it.

Thanks in advanced.

@pedrobrasileiro
Copy link

+1

3 similar comments
@pablozandona
Copy link

+1

@anandveld
Copy link

+1

@stvmachine
Copy link

+1

@Fmstrat
Copy link

Fmstrat commented Aug 7, 2018

We are past a year on this request, and we're on to Android Pie now. Any chance this could get assigned?

@imhoffd
Copy link
Contributor

imhoffd commented Aug 23, 2018

This PR in cordova-android will introduce a new syntax for adaptive icons: apache/cordova-android#448

@imhoffd imhoffd added this to Backlog 🤖 in Tooling 🔧 via automation Oct 9, 2018
@ludobit
Copy link

ludobit commented Oct 10, 2018

+1

@imhoffd
Copy link
Contributor

imhoffd commented May 15, 2019

Little update: our cordova-res utility now supports Android Adaptive Icons. It will also generate the config.xml syntax necessary for Cordova (support for which was recently added, so you will need latest Cordova and latest cordova-android).

cordova-res is becoming the default utility for generating Cordova resources in Ionic CLI 5.0.0, so this issue will be closed when 5.0.0 is released. See #3915.

Feel free to create issues in the cordova-res repo or reply here. Looking forward to some feedback!

@imhoffd imhoffd modified the milestones: CLI Future, CLI 5.0.0 May 15, 2019
@eikooc
Copy link
Contributor Author

eikooc commented May 21, 2019

Hi @dwieeb that looks awesome. I can see you included an option to set a color for the background, could you also make it possible to set a color for the foreground if it is a png with alpha channel?

@imhoffd
Copy link
Contributor

imhoffd commented May 21, 2019

@eikooc Is there a link to some documentation you could send me? As long as Cordova supports it, I feel comfortable supporting it as well in cordova-res.

@imhoffd
Copy link
Contributor

imhoffd commented May 29, 2019

Ionic CLI 5 is out!

The default for ionic cordova resources is now to use the cordova-res utility, which supports Android adaptive icons. Feels good to finally be able to close this issue. 😄

There is an issue for improving the documentation for this (I realize it's a bit lacking): ionic-team/ionic-docs#721

Please give the new CLI a try! Leave feedback/bug reports in either repo. Thanks!

@imhoffd imhoffd closed this as completed May 29, 2019
Tooling 🔧 automation moved this from Backlog 🤖 to Done 🎉 May 29, 2019
@ionic-team ionic-team locked and limited conversation to collaborators May 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Tooling 🔧
  
Done 🎉
Development

No branches or pull requests