Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(anilist): change text-bright, cleanup #1458

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

AnubisNekhet
Copy link
Member

Changes default value of --color-text-bright, removes redundant styling

@github-actions github-actions bot added the anilist AniList, AniChart label Nov 15, 2024
@AnubisNekhet AnubisNekhet marked this pull request as draft November 15, 2024 20:29
Comment on lines +28 to +37
// Scrollbar color
* {
scrollbar-color: @catppuccin[@@lightFlavor][@@accentColor]
fade(@catppuccin[@@darkFlavor][@crust], 0.2);
&:has(.site-theme-dark) {
scrollbar-color: @catppuccin[@@darkFlavor][@@accentColor]
fade(@catppuccin[@@darkFlavor][@crust], 0.2);
}
}

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm not completely sure if this will be fine performance-wise, but it's the only way I could style the scrollbars.

Copy link
Member

Choose a reason for hiding this comment

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

Why does it need to be applied to every single element? And why is this not in the #catppuccin mixin?

Copy link
Member Author

Choose a reason for hiding this comment

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

The userstyle uses the body element to select dark and light flavors, which unfortunately results in the scrollbar (Which is styled using the * selector) not being themed. It's the only way I could figure out a workaround, however I'm open to any changes.

@AnubisNekhet AnubisNekhet marked this pull request as ready for review November 15, 2024 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
anilist AniList, AniChart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants