development/vscode
development/vscode
1
0
Fork 0
mirror of https://github.com/Microsoft/vscode synced 2024-08-27 21:09:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #211236 from microsoft/tyriar/211220

Browse source 
Remove blank icon as option for terminal icon
This commit is contained in:
Daniel Imms 2024-04-24 03:24:39 -07:00 committed by GitHub
parent 76cc0e1712 d84506ea73
commit 3bec854a46
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/vs/workbench/contrib/terminal/browser/terminalIconPicker.ts
View file

@ -5,6 +5,7 @@
import { Dimension, getActiveDocument } from 'vs/base/browser/dom';
import { HoverPosition } from 'vs/base/browser/ui/hover/hoverWidget';
import { codiconsLibrary } from 'vs/base/common/codiconsLibrary';
import { Lazy } from 'vs/base/common/lazy';
import { Disposable } from 'vs/base/common/lifecycle';
import type { ThemeIcon } from 'vs/base/common/themables';
@ -18,6 +19,9 @@ const icons = new Lazy<IconContribution[]>(() => {
const iconDefinitions = getIconRegistry().getIcons();
const includedChars = new Set<string>();
const dedupedIcons = iconDefinitions.filter(e => {
if (e.id === codiconsLibrary.blank.id) {
return false;
}
if (!('fontCharacter' in e.defaults)) {
return false;
}