profile.colorTable is broken #5741

Open
opened 2026-01-31 00:20:26 +00:00 by claunia · 0 comments
Owner

Originally created by @DHowett-MSFT on GitHub (Dec 31, 2019).

Originally assigned to: @DHowett-MSFT on GitHub.

Setting a colorTable in a profile was broken in microsoft/terminal@8ba8f35dc5 when we set the default _schemeName to Campbell. During the construction of a TerminalSettings, we apply the table and then the scheme. If we do it in a different order, or delete _schemeName when we find colorTable, it works fine.

However: nobody noticed. This is the perfect opportunity to kill that feature or replace it with "colorTable overlays on top of the scheme"!

Related: #1069.

Originally created by @DHowett-MSFT on GitHub (Dec 31, 2019). Originally assigned to: @DHowett-MSFT on GitHub. Setting a `colorTable` in a profile was broken in microsoft/terminal@8ba8f35dc5 when we set the default `_schemeName` to `Campbell`. During the construction of a `TerminalSettings`, we apply the table and _then_ the scheme. If we do it in a different order, or delete `_schemeName` when we find `colorTable`, it works fine. _However:_ nobody noticed. This is the perfect opportunity to _kill that feature_ or _replace it with "`colorTable` overlays on top of the scheme"_! Related: #1069.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#5741