[1.12] Setting opacity as profiles.defaults doesn't work #15375

Closed
opened 2026-01-31 04:36:45 +00:00 by claunia · 1 comment
Owner

Originally created by @zadjii-msft on GitHub (Sep 28, 2021).

Originally assigned to: @zadjii-msft on GitHub.

discovered in the 1.12 bug bash on 9/28

"profiles": 
    {
        "defaults": 
        {
            "opacity": 90
        },
        "list": 
        [
            {
                "commandline": "powershell.exe",
                "experimental.retroTerminalEffect": true,
                "guid": "{61c54bbd-c2c6-5271-96e7-009a87ff44bf}",
                "hidden": false,
                "name": "Windows PowerShell",
                "opacity": 10,
                "unfocusedAppearance": 
                {
                    "colorScheme": "Solarized Light"
                }
            },
            {
                "commandline": "cmd.exe",
                "guid": "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}",
                "hidden": false,
                "name": "Command Prompt"
            },

CMD should be 90% opaque, it's 100%. We're thinking this was a bad merge, since other appearance settings work in the defaults.

Probably regressed in #11184

Originally created by @zadjii-msft on GitHub (Sep 28, 2021). Originally assigned to: @zadjii-msft on GitHub. _discovered in the 1.12 bug bash on 9/28_ ```jsonc "profiles": { "defaults": { "opacity": 90 }, "list": [ { "commandline": "powershell.exe", "experimental.retroTerminalEffect": true, "guid": "{61c54bbd-c2c6-5271-96e7-009a87ff44bf}", "hidden": false, "name": "Windows PowerShell", "opacity": 10, "unfocusedAppearance": { "colorScheme": "Solarized Light" } }, { "commandline": "cmd.exe", "guid": "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}", "hidden": false, "name": "Command Prompt" }, ``` CMD should be 90% opaque, it's 100%. We're thinking this was a bad merge, since other appearance settings work in the `defaults`. Probably regressed in #11184
Author
Owner

@zadjii-msft commented on GitHub (Sep 28, 2021):

  • settings.baseLayerProfile = Profile::FromJson(defaultsObject); does find the opacity in the settings

Oh boy I bet it's this block

        _Opacity = appearance.HasOpacity() ?
                       appearance.Opacity() :
                       UseAcrylic() ?
                       .5 :
                       1.0;
@zadjii-msft commented on GitHub (Sep 28, 2021): * `settings.baseLayerProfile = Profile::FromJson(defaultsObject);` does find the opacity in the settings Oh boy I bet it's this block ```c++ _Opacity = appearance.HasOpacity() ? appearance.Opacity() : UseAcrylic() ? .5 : 1.0; ```
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#15375