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

[Android] Update native dialog theme #98890

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

syntaxerror247
Copy link
Contributor

@syntaxerror247 syntaxerror247 commented Nov 6, 2024

This PR switches the theme for Android native dialogs to Theme_DeviceDefault_Dialog_Alert (on API 22+) to better match the device’s default style. This makes the dialogs feel more consistent with the system theme, creating a smoother and more familiar experience for users.

Size comparission of old and new apk

Screenshot 2024-11-06 134415

Before and After images of native dialogs

Before this PR

before

After this PR, tested on a Xiaomi device

xiaomi_after

After this PR, tested on a Motorola device

motorola_after

@syntaxerror247 syntaxerror247 requested a review from a team as a code owner November 6, 2024 11:35
@AThousandShips AThousandShips changed the title [Android] update native dialog theme [Android] Update native dialog theme Nov 6, 2024
@AThousandShips AThousandShips added this to the 4.x milestone Nov 6, 2024
Copy link
Contributor

@m4gr3d m4gr3d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@syntaxerror247 Is it possible to set it through the theme instead, so that it's automatic for any UI element we create?

@syntaxerror247
Copy link
Contributor Author

syntaxerror247 commented Nov 6, 2024

Is it possible to set it through the theme instead, so that it's automatic for any UI element we create?

@m4gr3d Yes, I was trying to do that as well but encountered an issue with the navigation bar displaying a white color in light mode.

image

Edit: I just realised, I can manually set the system bar color to black in themes.xml.

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

Successfully merging this pull request may close these issues.

Update Android Dialog Theme Alert window on Android theme looks old
3 participants