Skip to content

Conversation

@gerau
Copy link
Contributor

@gerau gerau commented Feb 9, 2026

There is an issue with displaying icons -- icon-theme.json counts as a
separate icon theme, so other icons besides simf and wit didn't
displayed.

Also, I had removed activation of LSP for .json files, because it
works on all jsons, which is obviously not desired.

@gerau gerau requested a review from delta1 as a code owner February 9, 2026 15:09
@gerau
Copy link
Contributor Author

gerau commented Feb 9, 2026

cc @KyrylR

gerau added 2 commits February 9, 2026 17:13
There is an issue with displaying icons -- `icon-theme.json` counts as a
separate icon theme, so other icons besides `simf` and `wit` didn't
displayed.

Also, I had removed activation of LSP for `.json` files, because it
works on **all** jsons, which is not desired
@KyrylR
Copy link
Collaborator

KyrylR commented Feb 9, 2026

Did you test it locally?

@gerau
Copy link
Contributor Author

gerau commented Feb 9, 2026

Did you test it locally?

Yeah, it's working fine on my side and showing the default icons and the new ones for .simf and .wit.

If the issue persists after this fix, you should try changing the icon pack via Ctrl + Shift + P -> Preferences: File Icon Theme.

@KyrylR
Copy link
Collaborator

KyrylR commented Feb 10, 2026

Did you test it locally?

Yeah, it's working fine on my side and showing the default icons and the new ones for .simf and .wit.

If the issue persists after this fix, you should try changing the icon pack via Ctrl + Shift + P -> Preferences: File Icon Theme.

We should mention this in the CHANGELOG.md

Copy link
Collaborator

@KyrylR KyrylR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK b3f07cd

@KyrylR KyrylR merged commit 4c7c626 into BlockstreamResearch:master Feb 10, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants