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

Tabs: Coloring issues #8564

Closed
bpasero opened this issue Jun 30, 2016 · 4 comments
Closed

Tabs: Coloring issues #8564

bpasero opened this issue Jun 30, 2016 · 4 comments
Assignees
Labels
polish Cleanup and polish issue ux User experience issues verified Verification succeeded workbench-tabs VS Code editor tab issues
Milestone

Comments

@bpasero
Copy link
Member

bpasero commented Jun 30, 2016

Summary of collected issues for coloring in tabs

Proposed solution from @bgashler1 in #8587

@bpasero
Copy link
Member Author

bpasero commented Jul 1, 2016

@bgashler1 I pushed a variant of your commit (681e42f) to master for June with the following changes:

  • I also use mix-blend-mode for the ::before element so that the border blends nicely in other themes (not sure why you did not set it)?
  • I avoid the trick/hack you do with changing the tab size between 34-36px by using z-index for the ::before element. in your solution I noticed the active tab jumping a bit when getting active

There is one thing which I don't like is the dropfeedback color over tabs and tabs container in light theme. It used to be the same blue you get when dragging over the tree or editor area but now with the mixed blend mode, I cannot get this colour to work. It seems this CSS property is inherited like opacity and you cannot disable it? If you have ideas how to solve that, we can still push for it.

Maybe you could verify this across OS and themes.

@sandy081 sandy081 added verified Verification succeeded and removed verified Verification succeeded labels Jul 1, 2016
@sandy081
Copy link
Member

sandy081 commented Jul 1, 2016

Leaving this verification to @bgashler1 based on above comment

@bgashler1 bgashler1 added verified Verification succeeded and removed verification-needed Verification of issue is requested labels Jul 2, 2016
@bgashler1
Copy link
Contributor

bgashler1 commented Jul 2, 2016

Nice work @bpasero. Unfortunately, I can't think of a simple way around the inheritance problem. But it really didn't bother me because I can't drag a tab to the "Open Editors" anyway, which is where that blue color was used for list items that have active focus.

Some of the more complex solutions I suggested earlier, such as CSS custom properties, and the CSS 4 pollyfill library would certainly enable us to avoid using the mix-blend mode though and avoid the inheritance problem.

@bpasero
Copy link
Member Author

bpasero commented Jul 2, 2016

@bgashler1 well you can drag it over to any opened editor and there you would get the large area filled in blue. I will see if I can tweak the color at least a little bit in light theme.

@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
polish Cleanup and polish issue ux User experience issues verified Verification succeeded workbench-tabs VS Code editor tab issues
Projects
None yet
Development

No branches or pull requests

3 participants