Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unfocusedAppearance cursorShape doesn't work #10215

Open
joca-bt opened this issue May 26, 2021 · 7 comments
Open

unfocusedAppearance cursorShape doesn't work #10215

joca-bt opened this issue May 26, 2021 · 7 comments
Assignees
Labels
Area-Settings Issues related to settings and customizability, for console or terminal Area-TerminalControl Issues pertaining to the terminal control (input, selection, keybindings, mouse interaction, etc.) Help Wanted We encourage anyone to jump in on these. Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal.
Milestone

Comments

@joca-bt
Copy link

joca-bt commented May 26, 2021

Windows Terminal version (or Windows build number)

1.8.1444.0

Other Software

No response

Steps to reproduce

settings.json:

{
    ...
    "profiles":
    {
        "defaults":
        {
            ...
            "cursorShape": "filledBox",
            "unfocusedAppearance": {
                "cursorShape": "emptyBox"
            }
        },
        ...
    },
    ...
}

Open Windows Terminal and some other application (e.g. Explorer) and move to that application so that Windows Terminal is unfocused.

Expected Behavior

Cursor changes to empty box.

Actual Behavior

Cursor disappears which makes the setting useless.

@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels May 26, 2021
@zadjii-msft
Copy link
Member

Oh huh. Good point. I wonder what the best way of handling this is. Presumably, if someone has a cursorShape in unfocusedAppearance, then they want the cursor to be visible when the control is unfocused. But I wonder how hard it would be for the Terminal to actually determine we're in that case - IIRC, if there's not a cursorShape in unfocusedAppearance, then we'll silently fall back to the cursor shape in the normal Appearance object.

@zadjii-msft zadjii-msft added Area-Settings Issues related to settings and customizability, for console or terminal Area-TerminalControl Issues pertaining to the terminal control (input, selection, keybindings, mouse interaction, etc.) Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal. labels May 27, 2021
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label May 27, 2021
@zadjii-msft zadjii-msft added Help Wanted We encourage anyone to jump in on these. Needs-Tag-Fix Doesn't match tag requirements labels May 27, 2021
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label May 27, 2021
@zadjii-msft zadjii-msft added this to the Terminal v2.0 milestone May 27, 2021
@DHowett
Copy link
Member

DHowett commented May 27, 2021

I guess we could use the HasCursorShape "inheritance" method!

@zadjii-msft
Copy link
Member

But a TermControl only has an IControlAppearance, which doesn't have that method! And I don't feel like bringing Terminal.Settings.Model beneath Terminal.Control

@DHowett DHowett removed the Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting label May 27, 2021
@sdsalyer
Copy link

sdsalyer commented Sep 15, 2021

I was surprised this didn't work since it was called out in the release notes for 1.8.1444.0:

A profile can now specify how it will appear when it's not focused via the unfocusedAppearance settings.json key (#8392)

Supported settings: foreground, background, selectionBackground, cursorColor, cursorShape, cursorHeight, backgroundImage, colorScheme, backgroundImageOpacity, backgroundImageStretchMode, backgroundImageAlignment, experimental.retroTerminalEffect, and experimental.pixelShaderPath

I would also point out it seems to work in VS Code, though I haven't changed any settings there other than the theme:
image

@zadjii-msft zadjii-msft modified the milestones: Terminal v2.0, 22H2 Jan 4, 2022
@pwsandoval
Copy link

The first comment keeps happening to me, is there a way to fix it?

@zadjii-msft zadjii-msft modified the milestones: 22H2, Backlog Jul 5, 2023
@mightycoco
Copy link

I was looking for this, as well. 1.18.2822.0

@AmerM137
Copy link

Same problem here, version 1.21.3231.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Settings Issues related to settings and customizability, for console or terminal Area-TerminalControl Issues pertaining to the terminal control (input, selection, keybindings, mouse interaction, etc.) Help Wanted We encourage anyone to jump in on these. Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal.
Projects
None yet
Development

No branches or pull requests

8 participants