[PR #1627] [MERGED] Replace caption buttons with ones that look close to correct #24629

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/1627
Author: @DHowett-MSFT
Created: 6/26/2019
Status: ✅ Merged
Merged: 7/1/2019
Merged by: @DHowett-MSFT

Base: master ← Head: dev/duhowett/minmaxcloseme


📝 Commits (3)

  • 4568b85 Replace MinMaxControl's buttons with ones that look really close to reality
  • 3c2d33f Address PR Feedback
  • 77329d1 Add a proper high contrast theme

📊 Changes

2 files changed (+167 additions, -6 deletions)

View changed files

📝 src/cascadia/TerminalApp/MinMaxCloseControl.cpp (+2 -0)
📝 src/cascadia/TerminalApp/MinMaxCloseControl.xaml (+165 -6)

📄 Description

Summary of the Pull Request

This pull request contains a bunch of XAML that makes our caption buttons look like this:

state image
normal image
hovered image
clicked image
maximized image

instead of like the characters _, - and X from Segoe UI.

References

Part of #1625.

PR Checklist

  • CLA signed. If not, go over here and sign the CLA
  • Tests added/passed/N/A
  • I've discussed this with core contributors already.

🔄 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/1627 **Author:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Created:** 6/26/2019 **Status:** ✅ Merged **Merged:** 7/1/2019 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `dev/duhowett/minmaxcloseme` --- ### 📝 Commits (3) - [`4568b85`](https://github.com/microsoft/terminal/commit/4568b8561baf91444f05ae79b8b964605c70e42a) Replace MinMaxControl's buttons with ones that look really close to reality - [`3c2d33f`](https://github.com/microsoft/terminal/commit/3c2d33f827cf1a4d26e3b70f1d59e9bcb0b1c6d6) Address PR Feedback - [`77329d1`](https://github.com/microsoft/terminal/commit/77329d1d14cc1917faec859af979059c27e04202) Add a proper high contrast theme ### 📊 Changes **2 files changed** (+167 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalApp/MinMaxCloseControl.cpp` (+2 -0) 📝 `src/cascadia/TerminalApp/MinMaxCloseControl.xaml` (+165 -6) </details> ### 📄 Description ## Summary of the Pull Request This pull request contains a bunch of XAML that makes our caption buttons look like this: |state|image| |-|-| |normal|![image](https://user-images.githubusercontent.com/14316954/60141892-55027f80-976c-11e9-8074-c8b77ca4a1d8.png)| |hovered|![image](https://user-images.githubusercontent.com/14316954/60141895-58960680-976c-11e9-89d2-08dbd664625f.png)| |clicked|![image](https://user-images.githubusercontent.com/14316954/60141898-5c298d80-976c-11e9-9a53-c9f394c3ca88.png)| |maximized|![image](https://user-images.githubusercontent.com/14316954/60141918-795e5c00-976c-11e9-83d4-fc0fe80acb57.png)| instead of like the characters `_`, `-` and `X` from Segoe UI. ## References Part of #1625. ## PR Checklist * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [x] Tests added/passed/N/A * [x] I've discussed this with core contributors already. --- <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:04:28 +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#24629