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

fix: less transparency #17

Closed
wants to merge 1 commit into from
Closed

fix: less transparency #17

wants to merge 1 commit into from

Conversation

lsunsi
Copy link

@lsunsi lsunsi commented Jul 11, 2024

I'm trying zed on linux (yay!) and immediately went for Dracula.
I was a little confused about the transparency and the issue #16 made me thing it was not intentional.

I made some local changes locally in order to keep exploring, but I have no idea if they are (1) correct (2) complete and especially if (3) they don't cause any side effects for other configurations.

I'm opening this PR just to share that these improved my situation and might help someone else.
I'd love to see this fixed upstream by an experienced maintainer :D

@lsunsi
Copy link
Author

lsunsi commented Jul 11, 2024

Other thing I noticed, when there's no tab open it's also fully transparent. I tried to ff the background transparency, but then it's just light purple when it probably should be a darker background pattern. I have no fix for this one for now, just wanted to mention.

@lsunsi
Copy link
Author

lsunsi commented Jul 11, 2024

Further, I'll say making just these changes is probably not good enough, because upon using it for a couple more hours I see that it does look a little weird. Maybe someone that understands about Dracula themes could make better changes and make this PR in the correct way, so I'll close it for now and keep my eyes peeled on the mentioned issue. Sorry for the confusion!

@lsunsi lsunsi closed this Jul 11, 2024
@lsunsi lsunsi deleted the less-transparency branch July 11, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant