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: set component styles again #12

Merged
merged 2 commits into from
Aug 16, 2024

Conversation

w-lfchen
Copy link
Contributor

@w-lfchen w-lfchen commented Aug 7, 2024

this pr re-adds the additions of #1 which were removed in #6.
i've also added the other options from the available component styles, trying to provide sensible options if needed; suggestions about color changes are very welcome.
closes #11 since nothing except the background is set to BrightBlack:

  • playback_metadata and playback_progress_bar are set according to the changes in Set components styles explicitly #1
  • playlist_desc is set to subtext0 since i found that to fit well

@w-lfchen w-lfchen mentioned this pull request Aug 7, 2024
@thebrrt
Copy link

thebrrt commented Aug 7, 2024

image

This is the way!

Copy link

@sgoudham sgoudham left a comment

Choose a reason for hiding this comment

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

Thanks for this! It definitely wasn't intentional to display less information.
I don't personally use this application but I think the choices you've made are sensible - I'll get around to trying it afterwards and see if improvements can be made but happy to merge for now!

@sgoudham sgoudham changed the title fix: set component styles explicitly again fix: set component styles again Aug 16, 2024
@sgoudham sgoudham merged commit 34b3d23 into catppuccin:main Aug 16, 2024
@w-lfchen w-lfchen deleted the fix/component-styles branch August 16, 2024 05:24
@w-lfchen
Copy link
Contributor Author

i just realized that the preview images are now out of date since the theme looks slightly different (the playback metadata is missing in the screenshots, compare them to the one posted in this pr's comments).
but that's probably not too important and i don't know how to update them, so i'll leave that for someone else ^^'

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.

Missing metadata
3 participants