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

Supporting themed app icons for Android 13 #1450

Closed
shanlin2dltk opened this issue Jun 29, 2022 · 6 comments · Fixed by #1550 or apache/cordova-docs#1285
Closed

Supporting themed app icons for Android 13 #1450

shanlin2dltk opened this issue Jun 29, 2022 · 6 comments · Fixed by #1550 or apache/cordova-docs#1285
Labels
Milestone

Comments

@shanlin2dltk
Copy link

Feature Request

In Android 13, one of the features is supporting themed app icons: https://developer.android.com/about/versions/13/features#themed-app-icons
Do we have a roadmap item to support those configuration in config.xml?

@breautek breautek added this to the 12.0.0 milestone Jul 16, 2022
@breautek
Copy link
Contributor

breautek commented Jul 16, 2022

Due to the volunteer nature of Apache projects, I cannot give any accurate timelines or roadmaps... but generally speaking, new API features are usually added in major releases.

If someone prepares a pull request and if it can be done without introducing breaking changes, then we can look at adding the feature to a minor release for current version 11.

For now, I'll add this to the version 12 milestone so we don't lose track.

@liyamahendra
Copy link
Contributor

@breautek I think I can fork the project and do necessary changes to have support for Themed Icons.

Quick questions:

  1. Sorry, I'm not aware - do we have Contribution Guidelines?
  2. How do you suggest I add this feature? Should I first share the changes I plan to do, get yours/someone from the Cordova team's inputs and then implement the changes in my fork or we (Myself & you or somebody from the Cordova team) work collaboratively on this?

Thanks!

@breautek
Copy link
Contributor

Hi @liyamahendra

The contributor guidelines can be found at https://cordova.apache.org/contribute/contribute_guidelines.html

If you have any high level questions, then it may be best to subscribe and create a thread on our dev mailing list. Otherwise sometimes the best way to collaborate is to build a PR so that we can review at the code level.

@liyamahendra
Copy link
Contributor

@breautek thank you for the details. I joined the dev mailing list. The Contribution guidelines says to have the issue being assigned by one of the committer prior to starting the work.

Would it be okay to have this assigned to me? Anything I need to do prior to this happens?

Thanks again.

@breautek
Copy link
Contributor

The Contribution guidelines says to have the issue being assigned by one of the committer prior to starting the work.

That's actually not a practice I've seen used much here in the Cordova community. The community here is pretty tight knit, so probably not necessary to have explicit assignments. I can see it being useful on larger communities to try to avoid several people working on the same thing. But I don't think that's a strict requirement here.

In short, go for it :D

On a side note, if you're interesting getting more involved, you may want to look into joining our slack channel. We used to use it for providing support but we have since moved to GitHub Discussions. However, we still use slack for realtime chat for some discussions. Dev mailing list is more or less for formal community decisions, as per Apache policies.

@liyamahendra
Copy link
Contributor

@breautek I created a PR to kick start the work. I added a brief description to the approach that I will be using:

image

Please comment when you can.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants