Skip to content

fix: inactive playlist top bar readability#84

Merged
justandras merged 4 commits intorjmunro/t-timers-extrafrom
justandras/fix/top-bar-readability
Apr 9, 2026
Merged

fix: inactive playlist top bar readability#84
justandras merged 4 commits intorjmunro/t-timers-extrafrom
justandras/fix/top-bar-readability

Conversation

@justandras
Copy link
Copy Markdown
Collaborator

@justandras justandras commented Apr 2, 2026

About the Contributor

This pull request is posted on behalf of the BBC.

Type of Contribution

This is a: Bug fix / UI improvement

Current Behavior

The dimmed top bar labels are hard to read when a playlist is not activated.
image
image
image

New Behavior

Use transparency for white color instead of hardcoding a grey color. I've also changed the clock color to white when deactivated.
image
image
image

Testing

  • I have added one or more unit tests for this PR
  • I have updated the relevant unit tests
  • No unit test changes are needed for this PR

Affected areas

This PR affects the top bar in the web ui.

Status

  • PR is ready to be reviewed.
  • The functionality has been tested by the author.
  • Relevant unit tests has been added / updated.
  • Relevant documentation (code comments, system documentation) has been added / updated.

@justandras justandras force-pushed the justandras/fix/top-bar-readability branch from 1475a8c to e480d44 Compare April 8, 2026 07:51
@jesperstarkar jesperstarkar self-requested a review April 9, 2026 09:34
Copy link
Copy Markdown
Collaborator

@jesperstarkar jesperstarkar left a comment

Choose a reason for hiding this comment

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

Skimmed the code, looks good to me.

Tested locally, works perfectly.

@justandras justandras merged commit 3238517 into rjmunro/t-timers-extra Apr 9, 2026
42 checks passed
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.

3 participants