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

Visual glitch: close tab button on dnd with shrink text #59063

Closed
usernamehw opened this issue Sep 20, 2018 · 5 comments
Closed

Visual glitch: close tab button on dnd with shrink text #59063

usernamehw opened this issue Sep 20, 2018 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities themes Color theme issues workbench-tabs VS Code editor tab issues

Comments

@usernamehw
Copy link
Contributor

Issue Type: Bug

Settings:

"workbench.editor.tabSizing": "shrink",
"tab.activeBackground": "#ff6a0015",
"tab.activeBorder": "#ff8f40",
"tab.activeForeground": "#61676c",
"tab.border": "#424d661a",
"tab.hoverBackground": "#ff6a0020",
"tab.inactiveBackground": "#fafafa",
"tab.inactiveForeground": "#9da2a6",
"tab.unfocusedActiveBorder": "#9da2a6",
"tab.unfocusedActiveForeground": "#9da2a6",
"tab.unfocusedInactiveForeground": "#9da2a6",

Trying to dnd tab:
screenshot 49

Probably the same is when srarting dnd in default dark+ theme:
screenshot 51

VS Code version: Code - Insiders 1.28.0-insider (af36149, 2018-09-20T14:09:20.109Z)
OS version: Windows_NT x64 10.0.17134

@bpasero
Copy link
Member

bpasero commented Sep 21, 2018

@Dari-K do you want to take this? It is possible that this is a regression from some work we did, though I cannot remember something during this milestone.

@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug help wanted Issues identified as good community contribution opportunities themes Color theme issues workbench-tabs VS Code editor tab issues labels Sep 21, 2018
@bpasero bpasero modified the milestone: September 2018 Sep 21, 2018
@bpasero
Copy link
Member

bpasero commented Sep 22, 2018

The second issue with the default dark theme is easy to fix, looks like this rule was competing:

.monaco-workbench > .part.editor > .content.dragged-over .editor-group-container.active > .title .tabs-container > .tab.sizing-shrink.dragged-over:not(.active):not(.dragged) > .tab-label::after,
.monaco-workbench > .part.editor > .content.dragged-over .editor-group-container > .title .tabs-container > .tab.sizing-shrink.dragged-over:not(.dragged) > .tab-label::after {
	background: linear-gradient(to left, ${adjustedColorDrag}, transparent) !important;
}

Adding a :not(.active) to the second rule's .editor-group-container fixes it. I am not sure about the other issue when using the rules as outlined in the comment.

bpasero added a commit that referenced this issue Sep 24, 2018
@Dari-K
Copy link
Contributor

Dari-K commented Sep 28, 2018

I'm actually having trouble reproducing it, I saw it once on Light+ just after I switched to it, but it stopped happening shortly afterwards and I haven't been able to get it to happen again.

@bpasero
Copy link
Member

bpasero commented Oct 1, 2018

Also do not see it anymore, closing.

@bpasero bpasero closed this as completed Oct 1, 2018
@usernamehw
Copy link
Contributor Author

Did you try to reproduce the first bug with the tab theming provided in the issue description?

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 15, 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 help wanted Issues identified as good community contribution opportunities themes Color theme issues workbench-tabs VS Code editor tab issues
Projects
None yet
Development

No branches or pull requests

3 participants