diff --git a/src/browser/Color.ts b/src/browser/Color.ts index a4e415af..32e311db 100644 --- a/src/browser/Color.ts +++ b/src/browser/Color.ts @@ -6,8 +6,6 @@ import { IColor } from 'browser/Types'; import { IColorRGB } from 'common/Types'; -// FIXME: Move Color.ts lib to common? - /** * Helper functions where the source type is "channels" (individual color channels as numbers). */ diff --git a/src/common/input/XParseColor.ts b/src/common/input/XParseColor.ts index 922bf8a9..8c023a38 100644 --- a/src/common/input/XParseColor.ts +++ b/src/common/input/XParseColor.ts @@ -49,7 +49,10 @@ export function parseColor(data: string): [number, number, number] | undefined { return result; } } - // FIXME: Once #3530 is resolved, implement named colors. + + // Named colors are currently not supported due to the large addition to the xterm.js bundle size + // they would add. In order to support named colors, we would need some way of optionally loading + // additional payloads so startup/download time is not bloated (see #3530). } // pad hex output to requested bit width