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

When selecting a new tab with the tab switcher, preview the selected tab in the Terminal #7409

Closed
zadjii-msft opened this issue Aug 26, 2020 · 4 comments · Fixed by #7796
Closed
Assignees
Labels
Area-User Interface 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. Priority-1 A description (P1) Product-Terminal The new Windows Terminal. Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release.
Milestone

Comments

@zadjii-msft
Copy link
Member

The new tab switcher is great. But I think it'd be even cooler if as I was switching tabs with it, the content of the terminal changed to reflect the tab I had selected.

So if I had [cmd, ubuntu, pwsh] open, with cmd focused, and opened the tab switcher to the "next tab", then the terminal would show me the ubuntu tab underneath the tab switcher. If I hit next tab again, then I want to see powershell underneath the switcher. Then, if I release the anchor key, just close the tab switcher. If I hit escape instead, then return me back to the tab I started on (cmd).

This is kinda a specialization of "Allow commands to be previewed in the command palette #6689", but this might be easier to do with the tab switcher, considering we already know that the original tab was (the state to restore to), and the command to use as the "preview".

I tried trivially implementing this today, but as I dispatched the commands from the cmdpal, the focus moved out of the cmdpal into the terminal, so that might be something to watch out for.

/cc @leonMSFT

@zadjii-msft zadjii-msft added Area-User Interface Issues pertaining to the user interface of the Console or Terminal Product-Terminal The new Windows Terminal. Issue-Task It's a feature request, but it doesn't really need a major design. Priority-2 A description (P2) labels Aug 26, 2020
@zadjii-msft zadjii-msft added this to the Terminal v2.0 milestone Aug 26, 2020
@ghost ghost added the Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting label Aug 26, 2020
@BartXZX
Copy link

BartXZX commented Aug 28, 2020

One of the reasons I switched back to the old switching behaviour is because I often have 5+ tabs open with the same name, and selecting a tab without actually seeing what is on it is hard.

@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 Aug 28, 2020
@nphmuller
Copy link

The lack of this feature is the main reason I disabled the tab switcher. I really like the switcher, but I like the previous behavior of seeing the content during "tabbing" better.

@zadjii-msft zadjii-msft added Priority-1 A description (P1) and removed Priority-2 A description (P2) labels Sep 29, 2020
@zadjii-msft zadjii-msft self-assigned this Oct 1, 2020
@ghost ghost added the In-PR This issue has a related PR label Oct 1, 2020
@ghost ghost closed this as completed in #7796 Oct 7, 2020
@ghost ghost added Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release. and removed In-PR This issue has a related PR labels Oct 7, 2020
ghost pushed a commit that referenced this issue Oct 7, 2020
## Summary of the Pull Request

![preview-ats-000](https://user-images.githubusercontent.com/18356694/94801728-18302a00-03ac-11eb-851d-760b92ebb46f.gif)

This PR enables the ATS to display the active tab as the user navigates the tab switcher. We do this by dispatching the tab switch actions as the user navigates the menu, and manually _not_ focusing the new tab when the tab switcher is open. 

## References

* #6732 - original tab switcher PR
* #6689 - That's a more involved, generic version of this, but this PR will be enough to stop most of the complaints hopefully


## PR Checklist
* [x] Closes #7409
* [x] I work here
* [ ] Tests added/passed
* [n/a] Requires documentation to be updated

## Validation Steps Performed

Opened tabs, tabbed through the menu, verified that it did what I'd expect
DHowett pushed a commit that referenced this issue Oct 19, 2020
## Summary of the Pull Request

![preview-ats-000](https://user-images.githubusercontent.com/18356694/94801728-18302a00-03ac-11eb-851d-760b92ebb46f.gif)

This PR enables the ATS to display the active tab as the user navigates the tab switcher. We do this by dispatching the tab switch actions as the user navigates the menu, and manually _not_ focusing the new tab when the tab switcher is open.

## References

* #6732 - original tab switcher PR
* #6689 - That's a more involved, generic version of this, but this PR will be enough to stop most of the complaints hopefully

## PR Checklist
* [x] Closes #7409
* [x] I work here
* [ ] Tests added/passed
* [n/a] Requires documentation to be updated

## Validation Steps Performed

Opened tabs, tabbed through the menu, verified that it did what I'd expect

(cherry picked from commit 22887d7)
@ghost
Copy link

ghost commented Nov 11, 2020

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

Handy links:

@ghost
Copy link

ghost commented Nov 11, 2020

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

Handy links:

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-User Interface 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. Priority-1 A description (P1) Product-Terminal The new Windows Terminal. Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants