After renaming tab with a long string it crashes or a blackscreen occures #11616

Closed
opened 2026-01-31 02:52:37 +00:00 by claunia · 7 comments
Owner

Originally created by @ghost on GitHub (Nov 27, 2020).

Windows 10 19042.620
Windows Terminal 1.4.3243.0
Windows Terminal Preview 1.5.3242.0

Steps to reproduce

Rename the Tab. Paste a long string

Expected behavior

No crash
No Blackscreen

Actual behavior

Windows Terminal Preview 1.5.3242.0:
Crash and Blackscreen

Windows Terminal 1.4.3243.0:
Blackscreen

image

Originally created by @ghost on GitHub (Nov 27, 2020). Windows 10 19042.620 Windows Terminal 1.4.3243.0 Windows Terminal Preview 1.5.3242.0 # Steps to reproduce Rename the Tab. Paste a long string # Expected behavior No crash No Blackscreen # Actual behavior Windows Terminal Preview 1.5.3242.0: Crash and Blackscreen Windows Terminal 1.4.3243.0: Blackscreen ![image](https://user-images.githubusercontent.com/67190487/100484612-fb655980-30b1-11eb-9582-a0c34c085fe8.png)
Author
Owner

@Don-Vito commented on GitHub (Nov 28, 2020):

@retfloww - after pasting the long title and getting to this weird screen, please try to hit Esc (alternatively, click outside of the terminal window so the Terminal window will loose focus, and then switch back to Terminal)

For me this fixes the screen in both in 1.4 in 1.5.
Though if you click the same title again, you will get the weird screen again.

What happens behind the scene is that there is a control for renaming, which is rendered incorrectly with long strings, but when it gets collapsed (via Esc or by loosing its focus) the weird UI is hidden.

@Don-Vito commented on GitHub (Nov 28, 2020): @retfloww - after pasting the long title and getting to this weird screen, please try to hit Esc (alternatively, click outside of the terminal window so the Terminal window will loose focus, and then switch back to Terminal) For me this fixes the screen in both in 1.4 in 1.5. Though if you click the same title again, you will get the weird screen again. What happens behind the scene is that there is a control for renaming, which is rendered incorrectly with long strings, but when it gets collapsed (via Esc or by loosing its focus) the weird UI is hidden.
Author
Owner

@ghost commented on GitHub (Nov 28, 2020):

Yes this actually works, thank you very much. Will this in any future be fixed?

@ghost commented on GitHub (Nov 28, 2020): Yes this actually works, thank you very much. Will this in any future be fixed?
Author
Owner

@Don-Vito commented on GitHub (Nov 28, 2020):

@retfloww - I am neither an expert nor from the team, but from what I learned about this community - everything gets solved at some point (unless it is an external dependency) ☺️

I will take a look at this later on to see if something catches my eye.

@Don-Vito commented on GitHub (Nov 28, 2020): @retfloww - I am neither an expert nor from the team, but from what I learned about this community - everything gets solved at some point (unless it is an external dependency) ☺️ I will take a look at this later on to see if something catches my eye.
Author
Owner

@Don-Vito commented on GitHub (Nov 28, 2020):

@retfloww - BTW, wondering what should be the correct behavior: do we really want to allow such long name, or should we instead truncate it / fail with a message.

Asking this because if truncation is an option, this is a one liner fix

@Don-Vito commented on GitHub (Nov 28, 2020): @retfloww - BTW, wondering what should be the correct behavior: do we really want to allow such long name, or should we instead truncate it / fail with a message. Asking this because if truncation is an option, this is a one liner fix
Author
Owner

@ghost commented on GitHub (Nov 28, 2020):

I would go for the second idea, I cannot imagine someone using a long tab name, there should be a cap for max. name lenght.
The Chrome browser handels renaming group tabs very well. I would like to see this same in the terminal.

@ghost commented on GitHub (Nov 28, 2020): I would go for the second idea, I cannot imagine someone using a long tab name, there should be a cap for max. name lenght. The Chrome browser handels renaming group tabs very well. I would like to see this same in the terminal.
Author
Owner

@zadjii-msft commented on GitHub (Nov 29, 2020):

Good job finding a really bizarre bug, and thanks @Don-Vito for hoping on a fix so quick!

@zadjii-msft commented on GitHub (Nov 29, 2020): Good job finding a really bizarre bug, and thanks @Don-Vito for hoping on a fix so quick!
Author
Owner

@ghost commented on GitHub (Jan 28, 2021):

:tada:This issue was addressed in #8433, which has now been successfully released as Windows Terminal Preview v1.6.10272.0.🎉

Handy links:

@ghost commented on GitHub (Jan 28, 2021): :tada:This issue was addressed in #8433, which has now been successfully released as `Windows Terminal Preview v1.6.10272.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.6.10272.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#11616