[PR #2443] [MERGED] SVG currently doesn't work; using .jpg instead #24932

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/2443
Author: @MikeTheGreat
Created: 8/15/2019
Status: ✅ Merged
Merged: 8/21/2019
Merged by: @DHowett-MSFT

Base: master ← Head: patch-1


📝 Commits (1)

  • 28b4468 SVG currently doesn't work; using .jpg instead

📊 Changes

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

View changed files

📝 doc/user-docs/UsingJsonSettings.md (+5 -4)

📄 Description

Summary of the Pull Request

Docs show example config with with a background image
That image is listed as being .SVG
SVG currently doesn't work
Let's have the docs use .jpg instead

Since the JPG won't stretch nicely we're also going to put it in the top-right corner without any scaling

References

We can go back to an SVG file after #2441 gets fixed

PR Checklist

  • Closes #xxx
  • 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.

Detailed Description of the Pull Request / Additional comments

No additional comments

Validation Steps Performed

I looked at it in the 'Preview' panel


🔄 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/2443 **Author:** [@MikeTheGreat](https://github.com/MikeTheGreat) **Created:** 8/15/2019 **Status:** ✅ Merged **Merged:** 8/21/2019 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`28b4468`](https://github.com/microsoft/terminal/commit/28b4468ac84a244d2459f8d222a54890a383f0e4) SVG currently doesn't work; using .jpg instead ### 📊 Changes **1 file changed** (+5 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `doc/user-docs/UsingJsonSettings.md` (+5 -4) </details> ### 📄 Description ## Summary of the Pull Request Docs show example config with with a background image That image is listed as being .SVG SVG currently doesn't work Let's have the docs use .jpg instead Since the JPG won't stretch nicely we're also going to put it in the top-right corner without any scaling ## References We can go back to an SVG file after #2441 gets fixed ## PR Checklist * [ ] Closes #xxx * [ ] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [X] 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. ## Detailed Description of the Pull Request / Additional comments No additional comments ## Validation Steps Performed I looked at it in the 'Preview' panel --- <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:06:11 +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#24932