[PR #5999] Correct the default 6x6x6 palette entries #26570

Closed
opened 2026-01-31 09:16:51 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/5999

State: closed
Merged: Yes


There is a range of 216 colors in the default 256-color table that is
meant to be initialized with a 6x6x6 color cube, with each color
component iterating over the values 00, 5F, 87, AF, D7, and
FF. A few of the entries incorrectly had the red component has DF,
when it should have been D7. This PR corrects those entries. It also
removes a bit of unnecessary whitespace in the first 100 entries.

Validation Steps Performed

I have a visual test script that renders the full 256-color palette,
using both the indexed color sequence (SGR 38;5) and the equivalent
rgb representation (SGR 38;2) side by side. Although the difference
was subtle when it was incorrect, I can now see that it has been fixed.

Closes #5994

**Original Pull Request:** https://github.com/microsoft/terminal/pull/5999 **State:** closed **Merged:** Yes --- There is a range of 216 colors in the default 256-color table that is meant to be initialized with a 6x6x6 color cube, with each color component iterating over the values `00`, `5F`, `87`, `AF`, `D7`, and `FF`. A few of the entries incorrectly had the _red_ component has `DF`, when it should have been `D7`. This PR corrects those entries. It also removes a bit of unnecessary whitespace in the first 100 entries. ## Validation Steps Performed I have a visual test script that renders the full 256-color palette, using both the indexed color sequence (`SGR 38;5`) and the equivalent rgb representation (`SGR 38;2`) side by side. Although the difference was subtle when it was incorrect, I can now see that it has been fixed. Closes #5994
claunia added the pull-request label 2026-01-31 09:16:51 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#26570