[PR #10774] [MERGED] Remove some unnecessary font features from our default feature list #28227

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/10774
Author: @PankajBhojwani
Created: 7/23/2021
Status: Merged
Merged: 7/26/2021
Merged by: @undefined

Base: mainHead: dev/pabhoj/default_feature_fix


📝 Commits (1)

📊 Changes

1 file changed (+1 additions, -8 deletions)

View changed files

📝 src/renderer/dx/DxFontRenderData.cpp (+1 -8)

📄 Description

Turns out, DWrite will automatically turn some features on even if they weren't included in the feature vector passed into it. Remove these features from our default list for easier readability.


🔄 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/10774 **Author:** [@PankajBhojwani](https://github.com/PankajBhojwani) **Created:** 7/23/2021 **Status:** ✅ Merged **Merged:** 7/26/2021 **Merged by:** [@undefined](undefined) **Base:** `main` ← **Head:** `dev/pabhoj/default_feature_fix` --- ### 📝 Commits (1) - [`460ca81`](https://github.com/microsoft/terminal/commit/460ca8116f6b2786af0aee3a8e481817a0d0d808) remove ### 📊 Changes **1 file changed** (+1 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `src/renderer/dx/DxFontRenderData.cpp` (+1 -8) </details> ### 📄 Description Turns out, DWrite will automatically turn some features on even if they weren't included in the feature vector passed into it. Remove these features from our default list for easier readability. --- <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:27:09 +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#28227