[PR #5592] [CLOSED] Prevent DXRenderer from using invalid properties #26376

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/5592
Author: @skyline75489
Created: 4/27/2020
Status: Closed

Base: masterHead: fix/font-size-crash-naive-fix


📝 Commits (1)

  • 1f94bd7 Prevent DXRenderer from using invalid properties

📊 Changes

1 file changed (+7 additions, -0 deletions)

View changed files

📝 src/renderer/dx/DxRenderer.cpp (+7 -0)

📄 Description

Summary of the Pull Request

The working fix, in case you needed it.

References

PR Checklist

  • Closes Refactor `UiaTextRange` (#5479)
  • CLA signed. If not, go over here and sign the CLA
  • Tests added/passed
  • Requires documentation to be 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

Validation Steps Performed

Manually validated.


🔄 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/5592 **Author:** [@skyline75489](https://github.com/skyline75489) **Created:** 4/27/2020 **Status:** ❌ Closed **Base:** `master` ← **Head:** `fix/font-size-crash-naive-fix` --- ### 📝 Commits (1) - [`1f94bd7`](https://github.com/microsoft/terminal/commit/1f94bd7cae315d901e9553f12d3699bda2ee5a6c) Prevent DXRenderer from using invalid properties ### 📊 Changes **1 file changed** (+7 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `src/renderer/dx/DxRenderer.cpp` (+7 -0) </details> ### 📄 Description <!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> ## Summary of the Pull Request The working fix, in case you needed it. <!-- Other than the issue solved, is this relevant to any other issues/existing PRs? --> ## References <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [X] Closes #5479 * [ ] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [ ] Requires documentation to be 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 <!-- Provide a more detailed description of the PR, other things fixed or any additional comments/features here --> ## Detailed Description of the Pull Request / Additional comments <!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well --> ## Validation Steps Performed Manually validated. --- <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:15:41 +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#26376