Skip to content

Apply correct text color to audio playback speed button#6223

Open
VelikovPetar wants to merge 1 commit intov7from
bug/fix_playback_speed_dark_mode
Open

Apply correct text color to audio playback speed button#6223
VelikovPetar wants to merge 1 commit intov7from
bug/fix_playback_speed_dark_mode

Conversation

@VelikovPetar
Copy link
Contributor

@VelikovPetar VelikovPetar commented Mar 6, 2026

Goal

The playback speed button text in audio record attachments was not using the correct theme color token, causing it to appear incorrectly in dark mode.

Implementation

Added color = ChatTheme.colors.controlPlaybackToggleText to the Text composable inside SpeedButton, so the label respects the theme color in both light and dark mode.

Testing

  • Open a message with an audio attachment
  • Toggle dark mode and verify the playback speed button text is visible and correctly colored in both themes

Summary by CodeRabbit

  • Bug Fixes
    • Fixed text color consistency in audio playback speed controls to properly align with the app's theme color scheme.

Co-Authored-By: Claude <noreply@anthropic.com>
@VelikovPetar VelikovPetar added the pr:bug Bug fix label Mar 6, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2026

PR checklist ✅

All required conditions are satisfied:

  • Title length is OK (or ignored by label).
  • At least one pr: label exists.
  • Sections ### Goal, ### Implementation, and ### Testing are filled.

🎉 Great job! This PR is ready for review.

@VelikovPetar VelikovPetar marked this pull request as ready for review March 6, 2026 14:52
@VelikovPetar VelikovPetar requested a review from a team as a code owner March 6, 2026 14:52
@coderabbitai
Copy link

coderabbitai bot commented Mar 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: f95e6b1f-cfef-4c40-b42e-ccc4bd338c84

📥 Commits

Reviewing files that changed from the base of the PR and between cf78e2e and d89507d.

📒 Files selected for processing (1)
  • stream-chat-android-compose/src/main/java/io/getstream/chat/android/compose/ui/attachments/content/AudioRecordAttachmentContent.kt

Walkthrough

Added a color parameter to the SpeedButton Text composable in AudioRecordAttachmentContent.kt to explicitly set the text color using ChatTheme.colors.controlPlaybackToggleText, ensuring consistent UI theming.

Changes

Cohort / File(s) Summary
Audio Record Attachment UI
stream-chat-android-compose/src/main/java/io/getstream/chat/android/compose/ui/attachments/content/AudioRecordAttachmentContent.kt
Added explicit color parameter to SpeedButton Text composable to apply ChatTheme.colors.controlPlaybackToggleText for consistent text color theming.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A splash of color, bright and true,
The SpeedButton now knows what hue to choose,
Theme consistency hops along,
Where playback toggles sing their song,
One little line, yet strong! 🎨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the primary change: applying the correct text color to the audio playback speed button.
Description check ✅ Passed The description covers the goal, implementation, and testing sections from the template, though UI change screenshots and contributor/reviewer checklists are incomplete.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bug/fix_playback_speed_dark_mode

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 and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2026

SDK Size Comparison 📏

SDK Before After Difference Status
stream-chat-android-client 5.25 MB 5.69 MB 0.44 MB 🟡
stream-chat-android-ui-components 10.60 MB 11.00 MB 0.40 MB 🟡
stream-chat-android-compose 12.81 MB 12.03 MB -0.78 MB 🚀

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

Labels

pr:bug Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants