Skip to content

fix: unknown user accent fallback [WPB-24378]#4774

Merged
Garzas merged 1 commit into
developfrom
fix/default-accent-color
Apr 30, 2026
Merged

fix: unknown user accent fallback [WPB-24378]#4774
Garzas merged 1 commit into
developfrom
fix/default-accent-color

Conversation

@Garzas
Copy link
Copy Markdown
Contributor

@Garzas Garzas commented Apr 30, 2026

https://wearezeta.atlassian.net/browse/WPB-24378


PR Submission Checklist for internal contributors

  • The PR Title

    • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
    • contains a reference JIRA issue number like SQPIT-764
    • answers the question: If merged, this PR will: ... ³
  • The PR Description

    • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

What's new in this PR?

Issues

Users who never selected an accent color, for example users who only used Web or iOS, could be shown with the current Android user's accent color in bubble message author styling.

Causes (Optional)

For users without an explicit accent color, Android receives or stores the accent as Accent.Unknown / accentId = 0. In bubble message author styling, Accent.Unknown fell back to the current theme primary color, which is based on the logged-in user's selected accent.

Solutions

Use the default Wire Blue accent for Accent.Unknown in bubble message author styling, while preserving the existing non-bubble fallback behavior. Also render name-based avatars with the default accent color mapping so unknown accents resolve consistently in light and dark mode.

@Garzas Garzas requested review from saleniuk and yamilmedina April 30, 2026 07:48
@Garzas Garzas self-assigned this Apr 30, 2026
@sonarqubecloud
Copy link
Copy Markdown

@Garzas Garzas added this pull request to the merge queue Apr 30, 2026
Merged via the queue into develop with commit 38fa9ce Apr 30, 2026
16 of 18 checks passed
@Garzas Garzas deleted the fix/default-accent-color branch April 30, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants