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

Indentation guides hard to see in darker themes #8410

Closed
4 tasks done
kieferrm opened this issue Jun 28, 2016 · 5 comments · Fixed by #11347
Closed
4 tasks done

Indentation guides hard to see in darker themes #8410

kieferrm opened this issue Jun 28, 2016 · 5 comments · Fixed by #11347
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug themes Color theme issues verified Verification succeeded

Comments

@kieferrm
Copy link
Member

kieferrm commented Jun 28, 2016

Testing #8232 on a MacBook Pro Retina display with brightness 2 steps below max while using a power adapter. This is my usual setting a regularly lit environments.

In the following themes the indentation guides should be changed to improve their visibility:

  • * Abyss
  • * High Contrast
  • * Monokai

In the following themes the indentation guides are subtle and easy to miss. However, changing those is optional.

  • * Dark (and Dark+)
@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug editor labels Jun 29, 2016
@alexdima
Copy link
Member

I am using the invisibles color from the theme, 100% the same as renderWhitespace. @aeschli owns the Dark and Dark+ themes (including the invisibles color):

https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/services/themes/electron-browser/themeService.ts#L385

@alexdima alexdima assigned aeschli and unassigned alexdima Jun 29, 2016
@aeschli
Copy link
Contributor

aeschli commented Jul 20, 2016

@bgashler1 Assigning to the UX team to decide on changes to the default colors.

@aeschli aeschli assigned bgashler1 and unassigned aeschli Jul 20, 2016
@alexdima alexdima added the themes Color theme issues label Aug 29, 2016
@alexdima alexdima added this to the August 2016 milestone Aug 29, 2016
@sandy081
Copy link
Member

sandy081 commented Aug 31, 2016

Some of the reported themes (Dark, Dark+, High Contrast) are owned by us. Others (Abyss, Monokai) are cloned and packaged along with VS Code. It means indirectly we own them and have to maintain them. So we should provide proper color for indent guides for all these themes.

@bgashler1 Can you please go through all our default packaged themes and come with a nice indentation guides color ?

Note: TM theme setting to use for identation guides is guide.

Also, as mentioned in description, except reported themes others look good to me

  • Abyss
  • Dark
  • Dark+
  • HighContrast
  • Monokai

@bgashler1
Copy link
Contributor

Reopening until PR is merged in

@sandy081
Copy link
Member

sandy081 commented Sep 2, 2016

Moving this to September milestone, since it is too late to make into August

@sandy081 sandy081 modified the milestones: September 2016, August 2016 Sep 2, 2016
@ramya-rao-a ramya-rao-a added the verified Verification succeeded label Sep 29, 2016
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug themes Color theme issues verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants