Skip to content

Conversation

@robertrossmann
Copy link
Contributor

@robertrossmann robertrossmann commented May 6, 2020

This fixes the editorGutter.foldingControlForeground colour contribution point by using the correct codicon names in the CSS.

Not sure when this regressed, I just noticed this while playing around with colour contributions and did not see any effect when I customised it.

robertrossmann added a commit to robertrossmann/vscode-remedy that referenced this pull request May 6, 2020
github-actions bot pushed a commit to robertrossmann/vscode-remedy that referenced this pull request May 6, 2020
# [4.13.0](4.12.0...4.13.0) (2020-05-06)

### Bug Fixes

* improve menu contrast by adding a border ([dd60676](dd60676))

### Features

* add Slack colour palette 😂 🤷‍♂️ ([8764188](8764188))
* customise folded line highlights and folding chevrons/buttons ([5e5a4a1](5e5a4a1)), closes [microsoft/vscode#97091](microsoft/vscode#97091)
* customise invalid list items' foreground 🤷‍♂️ ([98d9e40](98d9e40))
* improve consistency of focus foregrounds on tabs and breadcrumbs ([7f6a57a](7f6a57a))
@aeschli
Copy link
Contributor

aeschli commented May 8, 2020

The PR goes in the wrong direction.

The purpose of my change was to introduce new icon names ('folding-expanded','folding-collapsed') so that these can be themed individually by a product icon theme.
chevron-down/right are used at many places for various uses cases, not practical for themeing unless it's going too look like a chevron again.

So keep

const foldingExpandedIcon = registerIcon('folding-expanded', Codicon.chevronDown);
const foldingCollapsedIcon = registerIcon('folding-collapsed', Codicon.chevronRight);

and use foldingExpandedIcon.classNames or foldingExpandedIcon.cssSelector for wherever its class names are needed.

@robertrossmann
Copy link
Contributor Author

Thanks @aeschli for explanation - I pushed an updated version following your suggested approach. ✅

@aeschli aeschli merged commit 9b18ead into microsoft:master May 8, 2020
@aeschli
Copy link
Contributor

aeschli commented May 8, 2020

Perfect, thanks!

@aeschli aeschli added this to the May 2020 milestone May 8, 2020
@robertrossmann robertrossmann deleted the fix/folding-chevron-colour branch May 8, 2020 14:31
@github-actions github-actions bot locked and limited conversation to collaborators Jun 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

3 participants