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

Panel titles in Settings not styled properly #43568

Closed
mbenkmann opened this issue Feb 13, 2018 · 1 comment
Closed

Panel titles in Settings not styled properly #43568

mbenkmann opened this issue Feb 13, 2018 · 1 comment
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug settings-editor VS Code settings editor issues verified Verification succeeded

Comments

@mbenkmann
Copy link

  • VSCode Version: 1.20.0
  • OS Version: Linux

The Settings Window has 3 panels "DEFAULT SETTINGS", "USER SETTINGS" and "WORKSPACE SETTINGS". The title color of the "DEFAULT SETTINGS" panel seems to be hardcoded, which I guess is acceptable if not ideal.
The title color of "USER SETTINGS" and "WORKSPACE SETTINGS" is panelTitle.activeForeground when it is active, but a hardcoded color if inactive. Instead the inactive panel should follow panelTitle.inactiveForeground.

Furthermore, if contrastActiveBorder is unset by the user settings, the active panel's title is underlined. If contrastActiveBorder is set, even to a completely transparent color, the underline disappears. There does not seem to be a way to style that underline independently.

Both of the above are problems if you use the high contrast base theme but set the contrastActiveBorder to transparent in user settings because it's ugly and buggy (see different issue from me). It makes it so that the active panel's title cannot be underlined and the inactive panel's title cannot be set to a more subdued color. So it's hard to make obvious which of the 2 panels is selected.

To fix this:

  • the inactive panel title's color should obey panelTitle.inactiveForeground
  • the underline under the active panel title should have a color (which themes may set to transparent if desired) independent of contrastActiveBorder

Side note: The inactive panel titles in the PROBLEMS/OUTPUT/... side panel do obey panelTitle.inactiveForeground. The problem with the underline being tied to contrastActiveColor exists in that side panel, too, unfortunately.

@bpasero bpasero assigned sandy081 and unassigned bpasero Feb 14, 2018
@sandy081 sandy081 added bug Issue identified by VS Code Team member as probable bug settings-editor VS Code settings editor issues labels Sep 12, 2018
@sandy081 sandy081 added this to the September 2018 milestone Sep 12, 2018
@sandy081
Copy link
Member

You can change the underline colour using panelTitle.activeBorder.

Fixed inactive tabs in settings editor to use panelTitle.inactiveForeground

Verify that colour of inactive tabs in settings editor can be customised using panelTitle.inactiveForeground

@bpasero bpasero added the verified Verification succeeded label Sep 27, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 27, 2018
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 settings-editor VS Code settings editor issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants