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

[Megathread] Fullscreen & Focus Mode follow-up work #3486

Open
3 of 9 tasks
zadjii-msft opened this issue Nov 8, 2019 · 6 comments
Open
3 of 9 tasks

[Megathread] Fullscreen & Focus Mode follow-up work #3486

zadjii-msft opened this issue Nov 8, 2019 · 6 comments
Labels
Area-User Interface Issues pertaining to the user interface of the Console or Terminal Issue-Scenario Product-Terminal The new Windows Terminal.
Milestone

Comments

@zadjii-msft
Copy link
Member

zadjii-msft commented Nov 8, 2019

Taken from #531 (comment)

I'd suggest that this is broken into 3 implementation steps:

  1. Support fullscreen mode, with always hidden tabs
  2. Add a setting to have tabs always visible in fullscreen mode
  3. Enable "revealing" tabs on mouse hover near the top

Step 3 has some implementation details that are missing, like: on tabs reveal, should the tabs:

  • cover the terminal content, obscuring the top rows, but leaving the rest of the window unchanged
  • cause the terminal to resize
  • shift the terminal content down, obscuring the bottom rows of the terminal, but not forcing a resize?

Now that #3408 is merged (closing #531), we need some place to track the follow-up tasks I had outlined before:

Focus mode and Fullscreen share a lot of commonalities, so I'm grouping them into one thread for easy of x-linking.

Fullscreen

Focus Mode

@zadjii-msft zadjii-msft added Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. Area-User Interface Issues pertaining to the user interface of the Console or Terminal Product-Terminal The new Windows Terminal. labels Nov 8, 2019
@zadjii-msft zadjii-msft added this to the Terminal Backlog milestone Nov 8, 2019
@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 Nov 8, 2019
@miniksa miniksa removed the Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting label Nov 11, 2019
@nathpete-msft
Copy link
Member

nathpete-msft commented Dec 6, 2019

@zadjii-msft, as part of step 2, could that setting also have an option to only show tabs in fullscreen if there is currently more than one tab? Or would that setting just also take into account the existing alwaysShowTabs setting? The latter is how iTerm handles this, and I think feels pretty natural.

In that case, these settings might behave together like so:

State 2+ Tabs Always Show Tabs Show Tabs In Fullscreen Tabs Visible
Windowed Yes Yes N/A Yes
Windowed Yes No N/A Yes
Windowed No Yes N/A Yes
Windowed No No N/A No
Fullscreen Yes Yes Yes Yes
Fullscreen Yes Yes No No
Fullscreen Yes No Yes Yes
Fullscreen Yes No No No
Fullscreen No Yes Yes Yes
Fullscreen No Yes No No
Fullscreen No No Yes No
Fullscreen No No No No

@zadjii-msft
Copy link
Member Author

@nathpete-msft That seems pretty reasonable to me. If that's how iTerm2 does it, then that's a pretty good precedent.

@dominik-ba
Copy link
Contributor

dominik-ba commented Mar 25, 2020

I would love to have step 2 implemented. Especially if you have windows terminal in a second desktop you can simply switch to a fullscreen terminal with win + cmd + left/right. I know this works alreday but not knowing which tabs are open and which one I am using is a bit of a hassle.
That would be awesome! :)
Any plans on when this will get implemented?

PS: Now that I check on some more issues in this repo I'm confused since quite some people created (duplicate) issues about the padding of the tabs in fullscreen mode but I checked again and did not find a way or documentation how to enable this. Is this PR old or are those people not acutually talking about fullscreen?

@zadjii-msft
Copy link
Member Author

@dominik-ba This work is not currently on any particular milestone. I can guarantee that it's not making the cut for 1.0, since we're currently in a feature-freeze as we prepare to release 1.0.

I think some of the other issues you're referring too mainly mean "maximized" when they're saying "fullscreen"

@dominik-ba
Copy link
Contributor

Wouldn't it be relatively easy to enable th new toggle focus feature also on fullscreen?
This would check step 2 :)

@ghost ghost added In-PR This issue has a related PR and removed In-PR This issue has a related PR labels Apr 8, 2021
@zadjii-msft zadjii-msft modified the milestones: Terminal Backlog, Backlog Jan 4, 2022
@zadjii-msft zadjii-msft added Issue-Scenario and removed Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. labels Apr 4, 2022
@zadjii-msft zadjii-msft modified the milestones: Backlog, Megathreads Apr 4, 2022
@zadjii-msft zadjii-msft changed the title Fullscreen Mode follow-up work [Megathread] Fullscreen & Focus Mode follow-up work Apr 4, 2022
@GeekJosh
Copy link
Contributor

Just for visibility, I have opened a PR to address #11130 (no tabs in fullscreen): #18171

I really like the idea of #5677 (show tabs on hover), so once a decision is made on my current PR (adding the simpler true/false setting), I'm happy to take a look at implementing that next. As TabView doesn't currently support a "reveal on hover" setting, my initial thought would be to use to a Flyout to achieve that effect, though I will admit I don't have a huge amount of experience with WinUI so if that is a bad idea I'm open to looking in to alternative implementations if anyone has any suggestions?

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-Scenario Product-Terminal The new Windows Terminal.
Projects
None yet
Development

No branches or pull requests

6 participants