[PR #5847] Switch the Cascadia projects to til::color where it's easily possible to do so #26503

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

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

State: closed
Merged: Yes


This pull request moves swaths of Cascadia to use til::color for color
interop. There are still some places where we use COLORREF, such as in
the ABI boundaries between WinRT components.

I've also added two more til::color helpers - with_alpha, which takes
an existing color and sets its alpha component, and a
Windows::UI::Color convertor pair.

Future direction might include a TerminalSettings::Color type at the
idl boundary so we can finally stop using UInt32s (!) for color.

Validation Steps Performed

Tested certain fragile areas:

  • setting the background with OSC 11
  • setting the background when acrylic is in use (which requires
    low-alpha)
**Original Pull Request:** https://github.com/microsoft/terminal/pull/5847 **State:** closed **Merged:** Yes --- This pull request moves swaths of Cascadia to use `til::color` for color interop. There are still some places where we use `COLORREF`, such as in the ABI boundaries between WinRT components. I've also added two more til::color helpers - `with_alpha`, which takes an existing color and sets its alpha component, and a `Windows::UI::Color` convertor pair. Future direction might include a `TerminalSettings::Color` type at the idl boundary so we can finally stop using UInt32s (!) for color. ## Validation Steps Performed Tested certain fragile areas: * [x] setting the background with OSC 11 * [x] setting the background when acrylic is in use (which requires low-alpha)
claunia added the pull-request label 2026-01-31 09:16:27 +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#26503