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

add gruvbox inspired theme #29

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

Conversation

prashantlv
Copy link

I created this theme because I like gruvbox style and these colors are very pleasant to eyes. Please have a look and any advice or suggestions will be appreciated.

I did not try to copy it, but I created this theme because I like gruvbox style and these colors are very pleasant to eyes. Please have a look and provide any advice or suggestion.
@im-n1
Copy link

im-n1 commented Aug 24, 2021

  1. The filename has a typo
  2. here is sample screenshot
    image

The theme needs some tweak - currently playing song is almost invisible.

I'm using official gruvbox colors in my terminal.

@prashantlv
Copy link
Author

The file name changed correctly and the issue about the highlighted song that you are pointing at is with your terminal color, they aren't true.

@im-n1
Copy link

im-n1 commented Aug 27, 2021

I'm pretty sure my terminal is truecolor.

image

@im-n1
Copy link

im-n1 commented Aug 30, 2021

I did updated your theme and fixed some bugs. The theme now uses less colors so it's less disturbing.

image

Here is the theme file

# made by [email protected]
# best viewed on shaded or black terminal

background              = default       default
frame                   = yellow        default
window_title            = yellow        default
directory               = yellow        default
selected_directory      = black         cyan
playlist                = black         cyan 
selected_playlist       = red           default
file                    = cyan          default
selected_file           = black         cyan
marked_file             = yellow         default
marked_selected_file    = black         cyan
info                    = cyan          default
selected_info           = black         cyan
marked_info             = yellow         default
marked_selected_info    = black         cyan
status                  = yellow        default
title                   = cyan          default
state                   = yellow        default
current_time            = cyan          default
time_left               = cyan          default
total_time              = cyan          default
time_total_frames       = cyan          default
sound_parameters        = cyan          default
legend                  = cyan          default
disabled                = default       default   dim
enabled                 = yellow        default
empty_mixer_bar         = yellow        default
filled_mixer_bar        = black         yellow
empty_time_bar          = green         default
filled_time_bar         = black         yellow
entry                   = cyan          default
entry_title             = red           default
error                   = red           default
message                 = cyan          default
plist_time              = yellow        default

@prashantlv
Copy link
Author

That sounds great.

@im-n1
Copy link

im-n1 commented Aug 30, 2021

It took some time. It's not that easy to figure all theming possibilities.

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.

2 participants