Skip to content

fix(ui): metadata color on overlay - #2861

Open
renefloor wants to merge 1 commit into
fix/message-actions-backgroundfrom
fix/metadata-on-overlay
Open

fix(ui): metadata color on overlay#2861
renefloor wants to merge 1 commit into
fix/message-actions-backgroundfrom
fix/metadata-on-overlay

Conversation

@renefloor

@renefloor renefloor commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Submit a pull request

Linear: fixes FLU-647

CLA

  • I have signed the Stream CLA (required).
  • The code changes follow best practices
  • Code changes are tested (add some information if not applicable)

Description of the pull request

Needs: GetStream/stream-core-flutter#144

This updates the metadata to white in preview mode so it's still readable.

Screenshots / Videos

Before After
image image
image image

@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 3509c4a2-f394-4919-8211-d1b568526021

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@renefloor
renefloor changed the base branch from master to fix/message-actions-background August 1, 2026 08:15
Comment on lines -112 to +118
style: .from(textColor: colorScheme.accentPrimary, iconColor: colorScheme.accentPrimary),
style: .from(textColor: accentColor, iconColor: accentColor),

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@xsahil03x is it right that these are not really themeable? We do use StreamMessageItemTheme.of(context).annotation, but this textColor style overrides the theme. I guess we only do that when we expect the accentPrimary color, but still that one is not really themeable than right?

@renefloor
renefloor force-pushed the fix/metadata-on-overlay branch from b40ce47 to b6df593 Compare August 4, 2026 14:05
@renefloor
renefloor marked this pull request as ready for review August 4, 2026 14:10
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