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

[themes] Expose more styles for borders (editorBracketMatch.border) #46804

Closed
ghost opened this issue Mar 28, 2018 · 11 comments
Closed

[themes] Expose more styles for borders (editorBracketMatch.border) #46804

ghost opened this issue Mar 28, 2018 · 11 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) editor-bracket-matching Editor brace matching feature-request Request for new features or functionality themes Color theme issues
Milestone

Comments

@ghost
Copy link

ghost commented Mar 28, 2018

  • VSCode Version: 1.21.1
  • OS Version: macOS

Hi, currently you can set editorBracketMatch.border from all angles. My suggestion is to add the ability to border - bottom, top, right or left.

For e.g.

"editorBracketMatch.border": ["bottom", "#fff"]

I like, like it's in Atom where editorBracketMatch.border by default is to the bottom, I know we have HC themes and we need to keep it from all angles. But what about this idea?

@alexdima
Copy link
Member

This covers the problem of how to expose border customizations. Color is just one aspect, but then there are other aspects:

  • border style: dashed, dotted, solid
  • border width
  • having only one side of the border (like the OP mentions)
    etc.

@alexdima alexdima changed the title editorBracketMatch.border Expose more styles for borders (editorBracketMatch.border) Mar 28, 2018
@ghost
Copy link
Author

ghost commented Mar 28, 2018

@alexandrudima Amazing idea. This will reduce issues for other users because you can define these styles by yourself how you want it to look like. Also, I guess it's a good idea to let ppl decide how much px they want it to be.

So I believe something like this will be amazing (just brainstormed these):

"editorBracketMatch.border": ["1px", "solid", "bottom", "#fff"]
"editorBracketMatch.border": "1px solid bottom #fff"
"editorBracketMatch.border": {
  height: '1px',
  style: 'solid',
  position: 'bottom',
  color: '#fff'
}

@aeschli
Copy link
Contributor

aeschli commented Mar 28, 2018

I like

"editorBracketMatch.border": {
  height: '1px',
  style: 'solid',
  position: 'bottom',
  color: '#fff'
}

with all properties not listed use (solid, 1px, box) or the settings from the default/overwritten settings.

@usernamehw
Copy link
Contributor

usernamehw commented Mar 28, 2018

How should it look assigning different border color, style and width for sides?

Does this issue cover other borders like "editor.findMatchBorder" (example in this comment #30793 (comment)) ?
border top and bottom

@svipas
Copy link
Contributor

svipas commented Mar 28, 2018

@usernamehw I believe this should assign only to one side, though it can support array in position to apply for more sides.

"editorBracketMatch.border": {
  height: '1px',
  style: 'solid',
  position: ['bottom', 'top'], // array
  color: '#fff'
}

@usernamehw
Copy link
Contributor

So, different size/style/position/color would look like that?

"editorBracketMatch.border": {
  height: ['1px', '2px'],
  style: ['solid', 'dotted'],
  position: ['bottom', 'top'],
  color: ['#fff', '#ddd']
}

@ghost
Copy link
Author

ghost commented Mar 28, 2018

@usernamehw No, I see no point to use different border styles and colors for different positions.

@aeschli
Copy link
Contributor

aeschli commented Mar 28, 2018

I also would suggest to keep things simple and support a fixed set of border styles with single colour and width.

@aeschli aeschli added the feature-request Request for new features or functionality label Apr 18, 2018
@aeschli aeschli added this to the Backlog milestone Apr 18, 2018
@aeschli aeschli added the themes Color theme issues label Apr 18, 2018
@aeschli aeschli changed the title Expose more styles for borders (editorBracketMatch.border) [themes] Expose more styles for borders (editorBracketMatch.border) Apr 18, 2018
@timdmackey
Copy link

Glad to see this addresses my previous issue! This theme customization will be really welcome.

@kanlukasz
Copy link

@alexdima alexdima added the editor-bracket-matching Editor brace matching label Aug 5, 2019
@alexdima
Copy link
Member

Let's track in #19534

@alexdima alexdima added the *duplicate Issue identified as a duplicate of another issue(s) label Oct 16, 2019
@github-actions github-actions bot locked and limited conversation to collaborators Apr 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) editor-bracket-matching Editor brace matching feature-request Request for new features or functionality themes Color theme issues
Projects
None yet
Development

No branches or pull requests

6 participants