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

Contrast border on first split view #26113

Closed
pouyakary opened this issue May 6, 2017 · 5 comments
Closed

Contrast border on first split view #26113

pouyakary opened this issue May 6, 2017 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug themes Color theme issues verified Verification succeeded
Milestone

Comments

@pouyakary
Copy link
Contributor

pouyakary commented May 6, 2017

  • VSCode Version: Latest and greatest
  • OS Version: Sierra

When you set contrast borders if you look closely they have this 1pt of pixel not-perfectness:

screen shot 2017-05-06 at 2 35 09 pm

screen shot 2017-05-06 at 2 41 08 pm

Also, headers get not borders at right which is also problematic:
screen shot 2017-05-06 at 2 42 10 pm

@bpasero bpasero self-assigned this May 7, 2017
@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug themes Color theme issues labels May 7, 2017
@bpasero bpasero added this to the May 2017 milestone May 7, 2017
@bpasero
Copy link
Member

bpasero commented May 7, 2017

@pmkary can you share your theme changes so that I can see what happens to the header? I cannot reproduce in my simple test case:

"workbench.colorCustomizations": {
	"contrastBorder": "#ff0000"
}

image

The border of the entire sidebar should actually separate the header too.

I see the issue with the 1px layout glitch, good catch. Not sure if easily to fix though...

@pouyakary
Copy link
Contributor Author

pouyakary commented May 7, 2017

@bpasero Thanks a lot for your investigation :D

1

I can actually see that your editor has just the same problem, Look:
screen shot 2017-05-07 at 10 59 58 pm

Before the age of workbench configs I used to manually change the CSS of the page and I know how to fix that, just check this snippet

but if you want to have more detail this the full settings:


    "workbench.colorCustomizations": {
        "editor.lineHighlightBackground": "#f2f2f2",
        "contrastBorder": "#e5e5e5",
        "peekViewEditor.background": "#E2EEFA",
        "peekViewResult.background": "#E2EEFA",
        "focusBorder": "#3778B7",
        "widget.shadow": "#cccccc",
        "editorGroup.border": "#E0E0E0",
        "editorGroupHeader.noTabsBackground": "#f1f1f1",
        "editorGroup.dropBackground": "#f1f1f1",
        "editorGroupHeader.tabsBackground": "#f1f1f1",
        "tab.inactiveBackground": "#f1f1f1",
        "tab.inactiveForeground": "#aaaaaa",
        "tab.activeForeground": "#C94824",
        "tab.border": "#f1f1f1",
        "editorWidget.background": "#f7f7f7",
        "panelTitle.activeBorder": "#B052A1",
        "titleBar.activeBackground": "#f1f1f1",
        "button.background": "#3778B7",
        "input.background": "#ffffff",
        "input.border": "#ffffff",
        "editorLineNumber.foreground": "#cccccc",
        "scrollbarSlider.background": "#DC8564",
        "scrollbarSlider.activeBackground": "#C94824",
        "scrollbarSlider.hoverBackground": "#C94824",
        "list.activeSelectionBackground": "#E2EEFA",
        "list.activeSelectionForeground": "#3778B7",
        "list.focusBackground": "#E2EEFF",
        "list.inactiveSelectionBackground": "#E2EEFA",
        "list.highlightForeground": "#3778B7",
        "inputOption.activeBorder": "#C94824",
        "activityBar.background": "#f1f1f1",
        "activityBar.foreground": "#aaaaaa",
        "activityBarBadge.background": "#C94824",
        "statusBar.background": "#3778B7",
        "statusBar.noFolderBackground": "#B052A1",
        "statusBar.debuggingBackground": "#C94824",
        "sideBarSectionHeader.background": "#f7f7f7",
        "sideBarTitle.foreground": "#A56416",
        "sideBar.background": "#f1f1f1"
    }

To be used with Kary Foundation - Light if you want the full idea.

2

About the one pixel glitch I have no idea what so ever. I think if you change the sidebar header's height it would be the simplest to fix

@bpasero bpasero closed this as completed in 7b7132c May 8, 2017
@bpasero
Copy link
Member

bpasero commented May 8, 2017

I pushed a fix. We never actually want a contrast border for the first split view (this does not fix the reported issue but avoids it altogether).

Remains the issue with the border-right.

@bpasero bpasero reopened this May 8, 2017
@bpasero bpasero changed the title UI Sizing: Contrast Border Lines Contrast border lines vanish for selected tree item or split header May 8, 2017
@pouyakary
Copy link
Contributor Author

@bpasero thanks a lot 👍
As always the fact that you guys are so kind and caring about the issues and user request is really awesome and appreciated. Thanks a lot :D

@bpasero
Copy link
Member

bpasero commented May 8, 2017

I decided to extract the other issue to #26218 so that we can close this one and verify it.

Verify:

  • you are not seeing a border on top of the first split view (explorer, debug) in HC theme. Only for N > 1.

@bpasero bpasero closed this as completed May 8, 2017
@bpasero bpasero changed the title Contrast border lines vanish for selected tree item or split header Contrast border on first split view May 8, 2017
@roblourens roblourens added the verified Verification succeeded label Jun 1, 2017
@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

No branches or pull requests

3 participants