[PR #4406] [MERGED] Fix the search box in High Contrast by using the right theme resources #25745

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/4406
Author: @DHowett-MSFT
Created: 1/30/2020
Status: Merged
Merged: 1/30/2020
Merged by: @undefined

Base: masterHead: dev/duhowett/compare_and_contrast


📝 Commits (1)

  • f9697dd Fix the search box in High Contrast by using the right theme resources

📊 Changes

1 file changed (+5 additions, -6 deletions)

View changed files

📝 src/cascadia/TerminalControl/SearchBoxControl.xaml (+5 -6)

📄 Description

We were overriding the button foreground and the placeholder foreground using our
own custom resource names. They didn't exist in HC.

Instead of making them exist in HC, I made us use and override the real resource
names. Those ones have HC colors defined by the platform!

Fixes #4393.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/4406 **Author:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Created:** 1/30/2020 **Status:** ✅ Merged **Merged:** 1/30/2020 **Merged by:** [@undefined](undefined) **Base:** `master` ← **Head:** `dev/duhowett/compare_and_contrast` --- ### 📝 Commits (1) - [`f9697dd`](https://github.com/microsoft/terminal/commit/f9697dd3f93cbdc3f27fd6a51a5e325bbb81a935) Fix the search box in High Contrast by using the right theme resources ### 📊 Changes **1 file changed** (+5 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalControl/SearchBoxControl.xaml` (+5 -6) </details> ### 📄 Description We were overriding the button foreground and the placeholder foreground using our own custom resource names. They didn't exist in HC. Instead of making them exist in HC, I made us use and override the real resource names. Those ones have HC colors defined by the platform! Fixes #4393. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:11:31 +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#25745