Skip to content

fix: Volume indicator text in light mode - #1045

Merged
PartyDonut merged 1 commit into
DonutWare:developfrom
aviadlevy:worktree-fix+volume-modal-text-color
Aug 9, 2026
Merged

fix: Volume indicator text in light mode#1045
PartyDonut merged 1 commit into
DonutWare:developfrom
aviadlevy:worktree-fix+volume-modal-text-color

Conversation

@aviadlevy

Copy link
Copy Markdown
Contributor

Pull Request Description

Sets an explicit white text color on the volume indicator overlay so its label is readable against the dark background.

The volume indicator's Text had no explicit style, so it inherited the default theme text color (dark), which was unreadable on the 0.85-alpha black background. This matches the brightness indicator, which already sets the text color to white. The icon already rendered correctly and is unchanged.

Issue Being Fixed

On Android, when increasing/decreasing the volume during video playback, the volume overlay's percentage text rendered dark-gray on black and was effectively unreadable. The volume icon was fine, and the brightness overlay (both icon and text) was already correct.

Screenshots / Recordings

Tested On

  • Android
  • Android TV
  • iOS
  • Linux
  • Windows
  • macOS
  • Web

Checklist

  • If a new package was added, did you ensure it works for all supported platforms? Is the package well maintained (no new package added)
  • Check that any changes are related to the issue at hand.

The volume indicator's Text had no explicit style, so it inherited the
default theme text color (dark), which was unreadable against the
0.85-alpha black background. Match the brightness indicator by setting
the text color to white. The icon already rendered correctly and is
unchanged.

@PartyDonut PartyDonut left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks!

@PartyDonut PartyDonut added the bug Something isn't working label Aug 9, 2026
@PartyDonut PartyDonut changed the title fix: make volume indicator text readable on dark background fix: Volume indicator text in light mode Aug 9, 2026
@PartyDonut
PartyDonut merged commit 2c19ab2 into DonutWare:develop Aug 9, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this to Done in Fladder Aug 9, 2026
@aviadlevy
aviadlevy deleted the worktree-fix+volume-modal-text-color branch August 9, 2026 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants