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

New tab in Windows Terminal is missing content #15811

Closed
konnorandrews opened this issue Aug 9, 2023 · 9 comments
Closed

New tab in Windows Terminal is missing content #15811

konnorandrews opened this issue Aug 9, 2023 · 9 comments
Labels
Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.

Comments

@konnorandrews
Copy link

Windows Terminal version

1.17.11461.0

Windows build number

10.0.22621

Other Software

rustup - all versions including newest https://rustup.rs/

Steps to reproduce

See rust-lang/rustup#3430 for reproduction steps.

Expected Behavior

rustup's installer has a setup menu it prints. This should be displayed when rustup-init.exe is run.

Example shown below is when rebuilding the rustup-init.exe binary to wait 5 seconds before outputting anything.
image

Actual Behavior

The menu does not appear and the content seems to be duplicated on top of it. Scrolling only shows a blank section at the top of the buffer.
image

@konnorandrews konnorandrews added Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Aug 9, 2023
@konnorandrews
Copy link
Author

This seems to happen only when a new tab/window is created. If I run rustup-init.exe from a powershell instance in Windows Terminal it works as expected.

@zadjii-msft
Copy link
Member

I bet this is #14512. Can you try Terminal Preview (v1.18), and set that as your Default Terminal, and try again/?

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Aug 9, 2023
@ryanavella
Copy link

I can confirm that the problem is not reproducible with Terminal Preview (v1.18).

Given that the rustup folks would probably like a workaround for v1.17 and earlier, is there a straightforward way for an application to detect the version of Windows Terminal it was spawned from?

@ryanavella
Copy link

Also I am struggling to find information on the usual release cycle for Terminal, any idea when 1.18 will be stabilized?

@zadjii-msft
Copy link
Member

I honestly wouldn't bother with a workaround for earlier versions. I'm not even sure there is one based off the way the bug manifested. It was a race between the terminal's size and the conpty size, but I'm not even sure that introducing a forced delay before emitting text would always fix this.

I think there was some discussion about servicing that back to 1.17, lemme necro that thread quick.

We've kinda moved to a quarterly ship cycle, so we're starting to button up 1.19 now. I'd guess that 1.18 becomes stable in about a month?

Ultimately, if it were my call, I'd say that this isn't worth the engineering effort to waorkaround, when there's a fix in hand that's just "update to a newer Terminal version". Which should happen automatically for most folks in a month.

@zadjii-msft zadjii-msft added Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing. Needs-Discussion Something that requires a team discussion before we can proceed and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Aug 15, 2023
@ryanavella
Copy link

The reason some want a workaround is that it is fundamentally an accessibility issue. We want to be able to tell Windows users, who have no prior programming experience, "just run rustup-init.exe and you are good to go." But instead some are coming to forums and chat servers and saying Rust isn't working after installing it (note: when the installer menu is trimmed to the first 30 lines, it looks as though it installed successfully)

So if there isn't a workaround for detecting the terminal version, that probably limits us to either a delay-based workaround, or adding an extra installation step to install Windows Terminal Preview. I'm fine with either, I'd just like to better understand our options.

@microsoft-github-policy-service
Copy link
Contributor

This issue has been marked as duplicate and has not had any activity for 1 day. It will be closed for housekeeping purposes.

@zadjii-msft
Copy link
Member

We discussed this a bit in team sync today - @DHowett is just going to cut a servicing release of 1.17 that includes that fix. Honestly, that's just the easiest solution here. There's literally no reason for y'all to waste engineering cycles trying to work around an already fixed bug. And it would probably be a collective lot of engineering effort.

Easier to just push an update to the Store 😉

@zadjii-msft
Copy link
Member

(to close the loop - Terminal 1.18 stable is out now, with the aforementioned fix)

@zadjii-msft zadjii-msft removed the Needs-Discussion Something that requires a team discussion before we can proceed label Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing.
Projects
None yet
Development

No branches or pull requests

3 participants