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

Add addons support #2473

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Add addons support #2473

wants to merge 2 commits into from

Conversation

patrickgold
Copy link
Member

@patrickgold patrickgold commented May 13, 2024

Related #2468

Please only give implementation feedback in this comment thread, for general feedback regarding the addons integration as planned comment in the related issue!


Due to the theme screen rework in conjunction with addons it has been decided that the adapt to app theming option will be entirely removed (which wasn't working anyways for like the last 2 stable versions). There are no plans for re-adding this feature again anytime soon. The following issues are closed as not-planned: #274 #399 #853 #1216 #1313 #1338

@patrickgold patrickgold added the area: settings Issues regarding the Settings UI, structure, etc. label May 13, 2024
@patrickgold patrickgold self-assigned this May 13, 2024
@TPS
Copy link

TPS commented May 14, 2024

Due to the theme screen rework in conjunction with addons it has been decided that the adapt to app theming option will be entirely removed (which wasn't working anyways for like the last 2 stable versions).

Possible to do via addon?

@patrickgold
Copy link
Member Author

Possible to do via addon?

No, due to the fact that this feature requires reading the xml colors of the target app and those are a mess nowadays due to Material 1 2 and 3 all being out in the wild, and together with apps using Jetpack Compose those colors often times do not match at all what the apps colors are, thus I removed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: settings Issues regarding the Settings UI, structure, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants