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

Separators are nearly invisible with One Dark and One Light #261

Open
gilbertohasnofb opened this issue Sep 7, 2019 · 3 comments
Open

Comments

@gilbertohasnofb
Copy link

Separators are nearly invisible with One Dark and One Light:

image

image

But work fine with virtually all other UI themes:

image

image

@gilbertohasnofb
Copy link
Author

Perhaps an idea of how to solve this would be to use full white and full black colours with added alpha to the separator top/down and left/right sides. The alpha should take care to make it adapt to any background colour.

@suda
Copy link
Collaborator

suda commented Sep 9, 2019

I played around with mix-blend-mode to make the separator always brighter than the background. screen mode does work but it doesn't look nice IMO:
Screenshot 2019-09-09 13 07 47
Screenshot 2019-09-09 13 07 35
Screenshot 2019-09-09 13 07 26

I think the best way is to add an override in the theme-one-dark-ui.less and create one for any other theme that might need it.

@simurai do you have any thoughts on this perhaps?

@gilbertohasnofb
Copy link
Author

Thank you for taking a look at this.

but it doesn't look nice IMO

That PR I had initially opened had virtually the same exact results, which is why I closed it; it's definitely better than the current One Dark and One Light result, but it makes all other UI themes look worse, transforming their separators into a single bright line without shadow.

I think the best way is to add an override in the theme-one-dark-ui.less and create one for any other theme that might need it.

I think that would be a good solution. From my tests, One Dark and One Light are the only ones showing this problem so having an override for them make sense.

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