-
-
Notifications
You must be signed in to change notification settings - Fork 551
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
Theme file not loading on macOS #2007
Comments
just tried and on master aswell on the version you use via brew exactly your theme changes work as expected. |
@extrawurst Is there any way to tell gitui that it should log which themes it loads and in which folder it looks? This would really help debugging this on my machine. |
please run with and share the log created. also helpful is checking if |
@maxhoffmann still an issue? any findings via the log option? |
@extrawurst unfortunately yes. there are no entries in the log file about loading a config. |
@maxhoffmann can you please try to build from source? in be10d90 I added logging theme loading errors to the logfile |
I experience a similar issue. The logging added in be10d90 does not seem sufficient to debug this issue, since it does not print WHICH theme is loaded. |
I can confirm the To reproduce:
The relevant line in the log file is
As you can see, gitui is complaining that the config is not valid (which it is correct because I broke it), however, it should complain that the theme |
i think i know where the confustion comes from. the provided parameter is a filename, not a path. right now you cannot change the path but only the filename that's loaded from the config path. plus it silently fallback the default theme file name if the provided one is not a valid filename in that config path |
please test the version in the PR #2077 |
I see, that explains why I wasn't able to change the theme, it was not in the config folder! However, #2077 doesn't work correctly, I don't get a log entry if I try to use a nonexistent theme. |
now with da610e7 it works for me: |
@cgahr @maxhoffmann I think this improves the diagnostics a lot, there is an argument to be made if one should not be able to define a full theme path. but that's a story for another day |
I can confirm, at da610e7 every works now correctly. I think you can close this issue. |
* better theme file handling * print all possible err of loading theme closes extrawurst#2007
Describe the bug
When I override the default theme or try to add a new one, it doesn’t affect
gitui
’s theme.To Reproduce
Steps to reproduce the behavior:
gitui
viabrew
~/.config/gitui/theme.ron
gitui
Expected behavior
I expect the theme to change.
Context (please complete the following information):
13.6.3
0.24.3
Additional context
It would help to see where themes are loaded from, in case gitui doesn’t use
~/.config/gitui/
. Trying to add a custom theme and loading it via-t
also doesn’t lead to any error. It silently fails, so I cannot debug where thegitui
loads the themes from.The text was updated successfully, but these errors were encountered: