-
Notifications
You must be signed in to change notification settings - Fork 8.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
profile.colorTable
is broken
#4091
Comments
@DHowett-MSFT I just tried to customize my |
Consensus among the team is that color schemes are better suited to this purpose: we never implemented individual color index overriding for tables overlaid on schemes, which would have been the only way to make them broadly useful. If you don’t mind my asking, why might you prefer setting the color table in the profile over having a color scheme? |
It is very possible that a scheme is what I'm after. I tried out a couple of the out-of-box ones like "One Half Dark" and "Solarized Dark", but there were minor tweaks I wanted to make as my I tried playing around with the |
Ok I have found a pretty easy path towards installing a bunch more color schemes. I'm not sure if I am just missing this documentation somewhere, but for any future people that are having an issue installing their own custom scheme here is what you do:
Voila! @DHowett please let me know if there is some other documentation I can point to that already states this information. It would be great if there were instructions on how to do this in the root level README.md of this repository, no? |
🎉This issue was addressed in #5190, which has now been successfully released as Handy links: |
Setting a
colorTable
in a profile was broken in 8ba8f35dc5 when we set the default_schemeName
toCampbell
. During the construction of aTerminalSettings
, we apply the table and then the scheme. If we do it in a different order, or delete_schemeName
when we findcolorTable
, 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.
The text was updated successfully, but these errors were encountered: