[PR #13512] Change AdjustIndistinguishableColors to an enum setting instead of a boolean setting #29604

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

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

State: closed
Merged: Yes


Summary of the Pull Request

AdjustIndistinguishableColors can now be set to:

  • Never: Never adjust the colors
  • Indexed: Only adjust colors that are part of the color scheme
  • Always: Always adjust the colors

References

#13343

PR Checklist

  • Closes #xxx
  • CLA signed. If not, go over here and sign the CLA
  • Tests added/passed
  • Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
  • Schema updated.
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

For legacy purposes, true and false map to Indexed and Never respectively

Validation Steps Performed

Setting still works

**Original Pull Request:** https://github.com/microsoft/terminal/pull/13512 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request `AdjustIndistinguishableColors` can now be set to: - `Never`: Never adjust the colors - `Indexed`: Only adjust colors that are part of the color scheme - `Always`: Always adjust the colors ## References #13343 ## PR Checklist * [ ] Closes #xxx * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [ ] Documentation updated. If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/terminal) and link it here: #xxx * [ ] Schema updated. * [x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx ## Detailed Description of the Pull Request / Additional comments For legacy purposes, `true` and `false` map to `Indexed` and `Never` respectively ## Validation Steps Performed Setting still works
claunia added the pull-request label 2026-01-31 09:35: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#29604