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

Current line and fully selected line are near indistinguishable #3

Open
imfranklin opened this issue Dec 16, 2020 · 5 comments
Open

Comments

@imfranklin
Copy link

Since meta.text.selected and meta.cursor have the same background colors, the only difference between the current line with no selection and a line that is fully selected is the few pixels of a gutter background having changed.

lines

The Coda Dracula theme (https://github.com/dracula/coda) used for current line what this Nova theme has defined for meta.gutter.selected — would you entertain a pull request setting meta.cursor and meta.gutter.selected to the same background color?

@bdielman
Copy link
Member

Thanks for adding this issue. I'm looking into this, but I'd like to add a few more fixes along with this to bring the theme closer to the Dracula spec. I'm reaching out to maintainers of other themes to get a consensus on how some elements should be handled. I'll have a fix for this issue in the next release.

@imfranklin
Copy link
Author

Appreciate it! I've just made the jump after years on Coda, and this was the only difference that seemed out of place. The one-line change of imfranklin/nova@9b2e8ae and it feels like home again.

@bdielman
Copy link
Member

One thing to note (and it's been a very long time since I've used Coda), it looks like the current line in the code is using #44475a (the selection color according to the spec), so I'm curious why you're seeing a different color in the editor. It was also used as the current line background in the Dracula theme for VSCode (until issues with how VSCode stacks colors made them change it in v1.5.0).

Coda: https://github.com/dracula/coda/blob/master/Dracula.sss#L27
VSCode: https://github.com/dracula/visual-studio-code/blob/v2.14.1/src/dracula.yml#L41

Like I mentioned, I'm reaching out to other maintainers of Dracula themes to make sure this follows the spec as I plan to get his integrated into their library. 🧛‍♂️

@imfranklin
Copy link
Author

A fair question. I was relying on experience, not spec, though. Here's the Coda Dracula next to my modified version of your Nova Dracula — both looking at my same file:

image

And worth noting, the README image for https://github.com/dracula/coda displays the current line behavior I've been staring at for years:

image

@bdielman
Copy link
Member

I'll definitely consider it. Thanks again for bringing up the issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants