Skip to content

Rework header#21

Merged
RadValentin merged 10 commits into
mainfrom
20-rework-header
May 8, 2026
Merged

Rework header#21
RadValentin merged 10 commits into
mainfrom
20-rework-header

Conversation

@RadValentin
Copy link
Copy Markdown
Owner

@RadValentin RadValentin commented May 6, 2026

TODO

  • Header doesn't overflow on mobile
  • Header has metal style
  • Ensure metal buttons have hover and active states
  • Redirect to homepage on empty search (might be already implemented)

@RadValentin RadValentin linked an issue May 6, 2026 that may be closed by this pull request
2 tasks
RadValentin and others added 2 commits May 7, 2026 17:33
@RadValentin RadValentin marked this pull request as ready for review May 7, 2026 15:26
@RadValentin RadValentin requested a review from Copilot May 7, 2026 15:26
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refreshes the frontend header/UI styling by introducing a new “LED display” color palette and a new reusable button system, then applying those styles across the header, player controls, and track badges.

Changes:

  • Replaced legacy --accent-* variables with --led-* variables and added additional typography/radius tokens.
  • Introduced new reusable button classes (.btn, .btn-metal, .btn-amber, .btn-dark) and migrated several components to use them.
  • Updated header layout/branding markup and styling, plus refined badge styling for Rosamerica/Dortmund labels.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
frontend/src/index.css Adds new design tokens and new .btn* + badge variants; updates global colors.
frontend/src/components/TrackItem.tsx Applies new badge variant classes for genre labels.
frontend/src/components/Player.tsx Migrates player control buttons from .button to .btn*.
frontend/src/components/Player.css Updates player background styling to new gradients.
frontend/src/components/PlayButton.css Updates play button colors to new --led-* variables.
frontend/src/components/Header.tsx Reworks logo markup and migrates search/API controls to .btn*.
frontend/src/components/Header.css Reworks header/search styling and adds responsive layout adjustments.
frontend/src/components/Filters.tsx Migrates RESET button to .btn btn-amber.
frontend/src/components/Filters.css Updates color variables from accent to LED palette for filters UI.
frontend/index.html Loads JetBrains Mono font used by new UI elements.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread frontend/src/index.css
Comment thread frontend/src/components/Filters.css Outdated
Comment thread frontend/src/components/Header.css Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

Comment thread frontend/src/index.css
Comment thread frontend/src/index.css
@RadValentin RadValentin merged commit 2b15e07 into main May 8, 2026
3 checks passed
@RadValentin RadValentin deleted the 20-rework-header branch May 8, 2026 13:11
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.

Rework header to be mobile-friendly

2 participants