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

bg_search & fg_search colors can't override #113

Closed
jsgv opened this issue Oct 16, 2021 · 8 comments · Fixed by #127
Closed

bg_search & fg_search colors can't override #113

jsgv opened this issue Oct 16, 2021 · 8 comments · Fixed by #127
Assignees
Labels
assigned Under progress. bug Something isn't working
Projects

Comments

@jsgv
Copy link

jsgv commented Oct 16, 2021

After the latest updates my colors override options is not working anymore. It was working correctly yesterday.

require('github-theme').setup({
    theme_style = 'dimmed',
    colors = {
        bg_search = '#dae84d',
        fg_search = '#000000',
    }
})

Stopped working after doing a :PackerSync. Also noticed some other options were no longer being respected.

How it looks after :PackerSync:

image

@jsgv
Copy link
Author

jsgv commented Oct 16, 2021

Works after reverting to older commit:

use {
    'projekt0n/github-nvim-theme',
    commit = '771ac5b'
}

image

Like I mentioned above, it also affects other options such as transparent and dark_float. My lua is not that strong so I'm sorry I can't contribute more.

@ful1e5 ful1e5 added the assigned Under progress. label Oct 16, 2021
@ful1e5 ful1e5 added this to To do in Workboard via automation Oct 16, 2021
@ful1e5 ful1e5 moved this from To do to In progress in Workboard Oct 16, 2021
@ful1e5 ful1e5 added the bug Something isn't working label Oct 16, 2021
@ful1e5 ful1e5 self-assigned this Oct 16, 2021
@ful1e5 ful1e5 closed this as completed in 84e18f2 Oct 16, 2021
ful1e5 added a commit that referenced this issue Oct 16, 2021
Workboard automation moved this from In progress to Done Oct 16, 2021
@jsgv
Copy link
Author

jsgv commented Oct 16, 2021

I tested the latest commit and I am still experiencing the same error. @ful1e5

@ful1e5 ful1e5 reopened this Oct 16, 2021
Workboard automation moved this from Done to In progress Oct 16, 2021
@academo
Copy link

academo commented Oct 18, 2021

Something happened with the recent commits as well. I use transparent=true to keep my background black and with the latest update this is not respected.

I went back to the same commit 771ac5b you describe and it is back to work.

ful1e5 added a commit to ful1e5/github-nvim-theme that referenced this issue Oct 26, 2021
Workboard automation moved this from In progress to Done Oct 26, 2021
@ful1e5 ful1e5 changed the title colors option not working bg_search & fg_search colors can't override Oct 26, 2021
@jsgv
Copy link
Author

jsgv commented Oct 26, 2021

Updated to latest. Looks like I can not change themes now:

require("github-theme").setup({
    theme_style = "light",
})

Above configuration stays dark.

image

@ful1e5
Copy link
Member

ful1e5 commented Oct 26, 2021

@jsgv check #116

@thvdveld
Copy link

Also updated to the latest and cannot set the theme to dark_default. When setting it, the theme I get is dark, but not the dark_default one. (vim.o.background is set to dark)

@ful1e5
Copy link
Member

ful1e5 commented Oct 26, 2021

create a new issue.

@thvdveld
Copy link

Will do! I maybe thought it had something to do with this, sorry for the trouble!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned Under progress. bug Something isn't working
Projects
Development

Successfully merging a pull request may close this issue.

4 participants