Skip to content

Fix in-app message dark mode pinning on Android 13+ (#1056)#1057

Open
AndrazP wants to merge 1 commit into
Iterable:masterfrom
AndrazP:fix/1056-in-app-prefers-color-scheme
Open

Fix in-app message dark mode pinning on Android 13+ (#1056)#1057
AndrazP wants to merge 1 commit into
Iterable:masterfrom
AndrazP:fix/1056-in-app-prefers-color-scheme

Conversation

@AndrazP
Copy link
Copy Markdown

@AndrazP AndrazP commented May 25, 2026

In-app HTML messages always rendered the prefers-color-scheme: dark variant regardless of the system theme on API 33+. Two fixes:

  • Switch DialogFragment style from Theme.AppCompat.NoActionBar (a dark theme: extends Theme.AppCompat, not Theme.AppCompat.Light) to Theme.AppCompat.DayNight.NoActionBar, so isLightTheme tracks the system mode.

  • Build the WebView with the dialog's themed context instead of Fragment.getContext(), which returns the host activity context. On API 33+ WebView derives prefers-color-scheme from the hosting theme's isLightTheme attribute, so the DayNight fix only takes effect when the WebView is constructed under the dialog context.

🔹 Jira Ticket(s) if any

/

✏️ Description

See #1056

In-app HTML messages always rendered the prefers-color-scheme: dark
variant regardless of the system theme on API 33+. Two fixes:

- Switch DialogFragment style from Theme.AppCompat.NoActionBar (a dark
  theme: extends Theme.AppCompat, not Theme.AppCompat.Light) to
  Theme.AppCompat.DayNight.NoActionBar, so isLightTheme tracks the
  system mode.

- Build the WebView with the dialog's themed context instead of
  Fragment.getContext(), which returns the host activity context. On
  API 33+ WebView derives prefers-color-scheme from the hosting
  theme's isLightTheme attribute, so the DayNight fix only takes
  effect when the WebView is constructed under the dialog context.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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

Successfully merging this pull request may close these issues.

1 participant