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

The unfocused background flashes briefly when switching between tabs #9799

Open
zadjii-msft opened this issue Apr 13, 2021 · 5 comments
Open
Labels
Area-Settings Issues related to settings and customizability, for console or terminal Area-UserInterface Issues pertaining to the user interface of the Console or Terminal Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-2 A description (P2) Product-Terminal The new Windows Terminal.
Milestone

Comments

@zadjii-msft
Copy link
Member

Windows Terminal version (or Windows build number)

1.8+

Other Software

No response

Steps to reproduce

slap

            "unfocusedAppearance": {
                "background": "#888888"
            }

in a profile

Expected Behavior

Maybe there's a way we can not switch to the unfocused appearance when the control is removed from the UI tree entirely? IIRC we were talking about doing a similar thing to prevent the DxRender from doing anything when the tab is inactive

Actual Behavior

Switching between tabs with the same profile briefly flashes the gray color as it fades to the active color. It doesn't look great :confu

@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 Apr 13, 2021
@zadjii-msft
Copy link
Member Author

Looks like we'll also pop the unfocused appearance whenever any sort of modal UI is opened within the terminal app. Case in point, the command palette, the new tab flyout, the window renamer

unfocused-appearance-modals-000

I dunno if we want to include this or make it its own bug. Figured it's kinda related, or could be part of the solution. In all these cases, the control is unfocused, yes, but the pane is still active. Maybe we should only transition to the unfocused appearance if the pane is in the active tab, and is inactive?

@zadjii-msft zadjii-msft added Area-Settings Issues related to settings and customizability, for console or terminal Area-UserInterface Issues pertaining to the user interface of the Console or Terminal Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-2 A description (P2) Product-Terminal The new Windows Terminal. labels Apr 14, 2021
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label Apr 14, 2021
@zadjii-msft zadjii-msft added this to the Terminal v2.0 milestone Apr 14, 2021
@DHowett
Copy link
Member

DHowett commented Apr 14, 2021

Hmm.. I think making it follow pane activeness will change two fundamental things about it:

  1. right now it only needs TerminalControl to know when to switch states (nothing in tab or pane)
  2. It will not display the unfocused appearance when the app loses focus, because the active pane doesn’t change

the first is ideological, the second likely goes against why people wanted it in the first place. Hmm.

@zadjii-msft
Copy link
Member Author

Okay then, to massage the idea: maybe we should only transition to the unfocused appearance if ((the pane is in the active tab AND the pane and is inactive) OR (the entire app is inactive))

@zadjii-msft zadjii-msft removed the Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting label May 13, 2021
@zadjii-msft zadjii-msft modified the milestones: Terminal v2.0, 22H2 Jan 4, 2022
@miroslav-suvada

This comment was marked as off-topic.

@zadjii

This comment was marked as off-topic.

@zadjii-msft zadjii-msft modified the milestones: 22H2, Backlog Jul 5, 2023
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-UserInterface Issues pertaining to the user interface of the Console or Terminal Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-2 A description (P2) Product-Terminal The new Windows Terminal.
Projects
None yet
Development

No branches or pull requests

4 participants