Feature Request: Support environment variables for icon #2636

Closed
opened 2026-01-30 23:00:47 +00:00 by claunia · 3 comments
Owner

Originally created by @alefranz on GitHub (Jul 8, 2019).

Summary of the new feature/enhancement

Using environment variables for icon path is currently not working.

Current behavior

"icon" : "%SystemRoot%\Installer\{90CBDC15-F4FE-49D2-A245-EC066B70845F}\PowerShellExe.ico"

cause the terminal to crash.

Expected behavior

This should work and be equivalent to

"icon" : "C:\\Windows\\Installer\\{90CBDC15-F4FE-49D2-A245-EC066B70845F}\\PowerShellExe.ico"
Originally created by @alefranz on GitHub (Jul 8, 2019). # Summary of the new feature/enhancement Using environment variables for icon path is currently not working. ## Current behavior ``` "icon" : "%SystemRoot%\Installer\{90CBDC15-F4FE-49D2-A245-EC066B70845F}\PowerShellExe.ico" ``` cause the terminal to crash. ## Expected behavior This should work and be equivalent to ``` "icon" : "C:\\Windows\\Installer\\{90CBDC15-F4FE-49D2-A245-EC066B70845F}\\PowerShellExe.ico" ```
Author
Owner

@zadjii-msft commented on GitHub (Jul 8, 2019):

I believe this is a subset of work that needs to be done in #1468, but it's atomic enough that someone could fix this without fixing the larger issue.

@zadjii-msft commented on GitHub (Jul 8, 2019): I believe this is a subset of work that needs to be done in #1468, but it's atomic enough that someone could fix this without fixing the larger issue.
Author
Owner

@DHowett-MSFT commented on GitHub (Jul 8, 2019):

We should almost certainly genercizise the environment string expander from profile/cascadaisettings.

@DHowett-MSFT commented on GitHub (Jul 8, 2019): We should almost certainly genercizise the environment string expander from profile/cascadaisettings.
Author
Owner

@ghost commented on GitHub (Aug 3, 2019):

:tada:This issue was addressed in #2050, which has now been successfully released as Windows Terminal Preview v0.3.2142.0.🎉

Handy links:

@ghost commented on GitHub (Aug 3, 2019): :tada:This issue was addressed in #2050, which has now been successfully released as `Windows Terminal Preview v0.3.2142.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v0.3.2142.0) * [Store Download](https://www.microsoft.com/store/apps/9n0dx20hk701?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#2636