Pixel shader config file #12324

Closed
opened 2026-01-31 03:12:22 +00:00 by claunia · 3 comments
Owner

Originally created by @amoldeshpande on GitHub (Jan 30, 2021).

I am aware this is looking that gift horse straight in the chompers, but I'd like to see a separate config file for pixel shaders.

My evil scheme is to extend one of the prompt configuration utilities (like oh-my-posh) to change the shader based on some external state . I mean, who doesn't want an actual dumpster fire with smoke in the terminal when they're working in the COBOL repo ?

I can see doing this now by changing the settings file, but it would be nice to be not screw up the entire profile if something goes wrong (race conditions with other things changing the file etc.)

Proposed technical implementation details (optional)

add a setting like experimental.pixelShaderStandalonePath to a file that contains just a filename (or if you want consistency, a json snippet like the current settings for pixel shaders)

Originally created by @amoldeshpande on GitHub (Jan 30, 2021). <!-- 🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨 I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING: 1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement. 2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement. 3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number). 4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement. 5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement. All good? Then proceed! --> I am aware this is looking that gift horse straight in the chompers, but I'd like to see a separate config file for pixel shaders. My evil scheme is to extend one of the prompt configuration utilities (like oh-my-posh) to change the shader based on some external state . I mean, who doesn't want an actual dumpster fire with smoke in the terminal when they're working in the COBOL repo ? I can see doing this now by changing the settings file, but it would be nice to be not screw up the entire profile if something goes wrong (race conditions with other things changing the file etc.) # Proposed technical implementation details (optional) add a setting like `experimental.pixelShaderStandalonePath` to a file that contains just a filename (or if you want consistency, a json snippet like the current settings for pixel shaders)
Author
Owner

@zadjii-msft commented on GitHub (Feb 1, 2021):

Hmm. This might sound like a crazy idea, but this could be an application for settings fragments (#7632). Like, you have just a single file that's

{
  "profiles": [
    {
      "extends": "{whatever the guid is for the profile in question}",
      "experimental.pixelShaderPath": "whatever.hlsl"
    }
  ]
}

then you could modify just that file, and have the Terminal hot-reload the settings when that file changes.

I guess, this isn't that much better than just modifying the settings.json file directly, but at least you could be relatively confident that no one else would be touching that file.

/cc @PankajBhojwani as an fyi

@zadjii-msft commented on GitHub (Feb 1, 2021): Hmm. This might sound like a crazy idea, but this could be an application for settings fragments (#7632). Like, you have just a single file that's ```jsonc { "profiles": [ { "extends": "{whatever the guid is for the profile in question}", "experimental.pixelShaderPath": "whatever.hlsl" } ] } ``` then you could modify just that file, and have the Terminal hot-reload the settings when that file changes. I guess, this isn't that much better than just modifying the `settings.json` file directly, but at least you could be relatively confident that no one else would be touching that file. /cc @PankajBhojwani as an fyi
Author
Owner

@amoldeshpande commented on GitHub (Feb 1, 2021):

Sounds good. I didn't much like the idea of a one-off config key myself.

@amoldeshpande commented on GitHub (Feb 1, 2021): Sounds good. I didn't much like the idea of a one-off config key myself.
Author
Owner

@zadjii-msft commented on GitHub (Feb 1, 2021):

Okay cool. That should be coming in 1.7 (along with docs how to use it)!

@zadjii-msft commented on GitHub (Feb 1, 2021): Okay cool. That should be coming in 1.7 (along with docs how to use it)!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#12324