Skip to content

fix: glassy dark theme#1924

Open
Nicolach5 wants to merge 23 commits into
jeffvli:developmentfrom
Nicolach5:development
Open

fix: glassy dark theme#1924
Nicolach5 wants to merge 23 commits into
jeffvli:developmentfrom
Nicolach5:development

Conversation

@Nicolach5
Copy link
Copy Markdown

This PR fixes both the issue with the lyrics module (#1917) and the queue (#1895)

After fix:
Screenshot_20260407_130130
Screenshot_20260407_130146

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
feishin Ready Ready Preview, Comment Apr 9, 2026 10:17pm

@Nicolach5
Copy link
Copy Markdown
Author

The first fix removed the glassy effect for the player bar, so i readded that and instead calculated the height of the right sidebar separately

@jeffvli
Copy link
Copy Markdown
Owner

jeffvli commented Apr 8, 2026

@pyxfluff

Copy link
Copy Markdown
Contributor

@pyxfluff pyxfluff left a comment

Choose a reason for hiding this comment

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

I don't think this is ready, it seems to be pushing up for me, at least on web.

Image

Additionally, the removal of the glassy playerbar isn't something you did and can be fixed with this:

.fs-native-scroll-area-module-scroll-area {
  height: 100%
}

@Nicolach5
Copy link
Copy Markdown
Author

I'll try to fix it tomorrow as I really like the theme.
About the glassy player bar, in my first edit I removed

.fs-main-content-module-main-content-body {
    height: 100vh;
} 

which is how the content goes underneath the playerbar, but it also affects the right sidebar, so I was fixing my mistake there

@Nicolach5 Nicolach5 requested a review from pyxfluff April 8, 2026 03:26
@Nicolach5 Nicolach5 marked this pull request as ready for review April 8, 2026 06:42
I rewrote it completely to work with all the possible layouts, plus some fixes for the album page and the home page. It could probably be done way better with less code, but from my testing both in linux desktop and web, I probably covered all cases where elements could go underneath the playerbar, plus some other fixes i figured while experimenting
@Nicolach5 Nicolach5 changed the title fix sidebar glassy dark theme fix: glassy dark theme Apr 9, 2026
@Nicolach5
Copy link
Copy Markdown
Author

As mentioned in #1887, it seems there were other issues causing the publish task to fail because of an OOM error in the macos build. I tested in my fork and this happened even without my changes, inconsistently failing to build.

@Nicolach5 Nicolach5 marked this pull request as ready for review April 9, 2026 13:02
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