-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
QOL: Paint terminal win32 host background so it looks like the launch profile? #5572
Comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@letmaik do you mean a gray window like this? |
This comment has been minimized.
This comment has been minimized.
Thanks for writing this all up! I think it would be a nice quality of life improvement if we were to paint the window in a color closer to the one it'll eventually be filled with. @dreadnaut your issue seems different, somehow... that titlebar doesn't look familiar to me (the standard Windows one is narrower, and the custom one we draw has different caption buttons ...) |
(name changed to encompass what I think would be best) |
I use Terminal with acrylic disabled, and
which means I only see the standard Windows titlebar until I open a second tab. If I switch to tabs-in-titlebar, the flash has the same colour. I'd say it's the same issue 🙂 |
I'm gonna also x-ref #11561 here. With the transparency changes in 1.12, the backgound starts as totally transparent, instead of gray/white. There's also now the possibility that the Terminal is launching:
So this isn't necessarily straighforward "use the default profile's BG to paint the window". |
In case this changes priority at all: getting flashbanged in the eyes like this is the number one thing I know of that makes dark themes unhelpful for some people - if your eyes are sensitive in the right way, light theme is bad and unpleasant but still better than to let your eyes get used to dark only to get sliced across the eyes with a momentary flash. |
For anyone looking for a slightly better experience, I noticed that if I install WSL Debian, set my default terminal application to Windows Terminal instead of Windows Console Host, and then run Debian, I only get the flash on close rather than on open. (I presume you'd get the same improvement with any command-line program which indirectly launches Windows Terminal instead of directly, but I haven't checked.) |
Actually I take that last statement back. Upon further testing it seems that it's only sometimes fast enough to not do the white-ish flash on launch in those cases, but other times I still get the flash. Also an interesting effect is that the initial color on start and final color on close are different. The open color seems dimmer, like a gray, while the close color seems closer to a white. |
You know what, like 8 months ago we started having the HWND be entirely hidden until the window is ready to render. That basically makes this issue moot 😄 |
Now that you mention it, I don't remember being bothered by this problem lately. (I moved to Emacs through gWSL as my daily driver terminal emulator about a year ago, so in those 8 months I've only opened the terminal a handful of times.) Checking now, I can confirm that this issue seems to no longer happen, on both open and close. Big thanks. |
Environment
Steps to reproduce
Start Terminal and observe the short gray-to-default-profile-background-color flicker before the shell is loaded. Close Terminal and observe the same in reverse.
Expected behavior
There should be no flickering. When starting, the background of the "empty" shell area should be the same color as the default profile. When closing, the shell area should use the background color of the currently active profile/shell.
Actual behavior
The "empty" shell area is always gray, which means it transitions very quickly from gray to (typically) black when the default profile shell is displayed. Similarly when closing Terminal.
The text was updated successfully, but these errors were encountered: