Skip to content

fix: Change navbar background to gradient#35

Merged
RadicalMuffinMan merged 2 commits into
Moonfin-Client:masterfrom
coredev-uk:patch-1
Mar 5, 2026
Merged

fix: Change navbar background to gradient#35
RadicalMuffinMan merged 2 commits into
Moonfin-Client:masterfrom
coredev-uk:patch-1

Conversation

@coredev-uk

Copy link
Copy Markdown
Contributor

Makes the background change more graceful for users with backdrops enabled.

Copilot AI review requested due to automatic review settings March 5, 2026 16:00
@RadicalMuffinMan RadicalMuffinMan merged commit b8ffcbf into Moonfin-Client:master Mar 5, 2026
2 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

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 improves the visual experience for users who have backdrop images enabled in the Jellyfin Moonfin plugin. Previously, when the navbar or sidebar was active, the page body was forced to a flat black background, which abruptly obscured any backdrop imagery. The change replaces the solid black background with a vertical gradient that fades from black (at the top, where the navbar/sidebar sits) to transparent (at the bottom), allowing backdrops to show through more gracefully.

Changes:

  • navbar.css: Body background changed from solid #000 to a linear-gradient(to bottom, black 60%, transparent) when the navbar is active.
  • sidebar.css: Same gradient applied to the body background when the sidebar is active, matching the navbar treatment.

Reviewed changes

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

File Description
frontend/src/styles/navbar.css Replaces flat black body background with a bottom-fading gradient when navbar is active
frontend/src/styles/sidebar.css Replaces flat black body background with a bottom-fading gradient when sidebar is active

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

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