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

Come up with a better solution for reloading tabTitle #3989

Closed
zadjii-msft opened this issue Dec 17, 2019 · 1 comment
Closed

Come up with a better solution for reloading tabTitle #3989

zadjii-msft opened this issue Dec 17, 2019 · 1 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-Task It's a feature request, but it doesn't really need a major design. Needs-Tag-Fix Doesn't match tag requirements Product-Terminal The new Windows Terminal.

Comments

@zadjii-msft
Copy link
Member

@DHowett-MSFT

tabTitle is the only one it will matter for (and it doesn't work right right now), but this will be lost on a settings reload.

@zadjii-msft

That wouldn't matter though, would it? The startingTitle doesn't work on reload anyways - it's just the title the process starts with.

@DHowett-MSFT

Er that's what I mean when i say "doesn't work right now" -- tabTitle is a weird one where it should totally win if suppressApplicationTitle is turned on, right? So we have a minor issue right now where the tab title never gets pushed down to the right place (Terminal Core right now) on reload.. but this compounds the issue b/c if we DO implement that properly profiles started with this shortcut will have their tab titles lost.

We need to shake this story out, esp. with #3825 merged. If a tab hasn't set the title, and it's still using the initial "tabTitle", then when we hot-reload, we should update that value. If someone's using a title that's provieded as a part of the tabTitle in NewTabArgs, we should also probably use that instead of the settings one on reload.

This also represents work to dig through this story and make sure these settings are really getting plumbed through to the right layers, since we're getting pretty consistent bugs with it.

@zadjii-msft zadjii-msft added Product-Terminal The new Windows Terminal. Issue-Task It's a feature request, but it doesn't really need a major design. labels Dec 17, 2019
@zadjii-msft zadjii-msft added this to the Terminal Backlog milestone Dec 17, 2019
@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 Dec 17, 2019
@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 labels Dec 17, 2019
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label Dec 17, 2019
@DHowett-MSFT DHowett-MSFT removed the Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting label Dec 19, 2019
@zadjii-msft
Copy link
Member Author

Meh, I'm going to say "no one's complained about this". Sure, we could see if a tab has the same title as a changed tabTitle, and then manually change it, but that seems like a lot of work that no one seems to be bothered by.

@ghost ghost added the Needs-Tag-Fix Doesn't match tag requirements label Dec 9, 2021
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-Task It's a feature request, but it doesn't really need a major design. Needs-Tag-Fix Doesn't match tag requirements Product-Terminal The new Windows Terminal.
Projects
None yet
Development

No branches or pull requests

3 participants