-
Notifications
You must be signed in to change notification settings - Fork 5
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
title_bar.inactive_background is missing #2
Comments
Ok brilliant, if you want you can submit a pr and I'll merge it in, or I can just do it myself! |
I found an issue reported to the Zed repo here: zed-industries/zed#14955 and they reference this issue: zed-industries/zed#15709 which is supposed to be in version 148 but it still doesn't work for me 🤔 so just to be safe I can create a PR! 🙌 |
I tried to make a branch and push but I am not allowed to :( This is my diff: diff --git a/themes/everforest-dark.json b/themes/everforest-dark.json
index e151ff2..50a1222 100644
--- a/themes/everforest-dark.json
+++ b/themes/everforest-dark.json
@@ -39,6 +39,7 @@
"icon.accent": null,
"status_bar.background": "#21272b",
"title_bar.background": "#21272b",
+ "title_bar.inactive_background": "#1b2024",
"toolbar.background": "#272e33",
"tab_bar.background": "#21272b",
"tab.inactive_background": "#21272b", |
@patrickswijgman do I need to change a setting in GitHub or something to allow pull requests? Or is it a problem on your end? |
It seems that the
title_bar.inactive_background
token for the theme is missing, resulting in a transparent title bar when inactive.I'm running Linux (Ubuntu) and to fix it I had to add this to my settings:
I really love this theme so thanks a lot for the port! Cheers!
The text was updated successfully, but these errors were encountered: