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

title_bar.inactive_background is missing #2

Open
patrickswijgman opened this issue Aug 15, 2024 · 4 comments
Open

title_bar.inactive_background is missing #2

patrickswijgman opened this issue Aug 15, 2024 · 4 comments

Comments

@patrickswijgman
Copy link

patrickswijgman commented Aug 15, 2024

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:

  "experimental.theme_overrides": {
    "title_bar.inactive_background": "#1b2024"
  },

I really love this theme so thanks a lot for the port! Cheers!

@ThomasAlban
Copy link
Owner

Ok brilliant, if you want you can submit a pr and I'll merge it in, or I can just do it myself!

@patrickswijgman
Copy link
Author

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! 🙌

@patrickswijgman
Copy link
Author

patrickswijgman commented Aug 15, 2024

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",

@ThomasAlban
Copy link
Owner

ThomasAlban commented Aug 28, 2024

@patrickswijgman do I need to change a setting in GitHub or something to allow pull requests? Or is it a problem on your end?
You should be able to fork this repository, make the change, then submit a PR by clicking contribute on the GitHub page of your forked repository

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

No branches or pull requests

2 participants