Make the TabView span the entire window width #14096

Closed
opened 2026-01-31 04:00:45 +00:00 by claunia · 4 comments
Owner

Originally created by @ghost on GitHub (Jun 3, 2021).

Description of the new feature/enhancement

The TabView currently stops after the add button. This is a problem because it has a shadow which gets interrupted.
It's harder to see with the default gray titlebar, but if you could change it to white it would be noticeable.

It would be ideal if the TabView would span the entire window.

tabviewcurrent
tabviewfixed

Proposed technical implementation details

It can be achieved by moving the drag bar and caption buttons inside the TabStripFooter.

Originally created by @ghost on GitHub (Jun 3, 2021). # Description of the new feature/enhancement The TabView currently stops after the add button. This is a problem because it has a shadow which gets interrupted. It's harder to see with the default gray titlebar, but if you could change it to white it would be noticeable. It would be ideal if the TabView would span the entire window. ![tabviewcurrent](https://user-images.githubusercontent.com/84711285/120685914-14a7f200-c4a9-11eb-8f80-8e38ecc09f9c.png) ![tabviewfixed](https://user-images.githubusercontent.com/84711285/120685920-15d91f00-c4a9-11eb-9318-e36599cb817f.png) # Proposed technical implementation details It can be achieved by moving the drag bar and caption buttons inside the TabStripFooter.
Author
Owner

@zadjii-msft commented on GitHub (Jul 6, 2021):

So this is a cool idea, but also fraught with peril. We're doing all sorts of wonky Win32 hacks to let the user drag that region as it is. I'd love to have the shadow work across that entire region, but we'll need to be careful to not break that behavior.

@zadjii-msft commented on GitHub (Jul 6, 2021): So this is a cool idea, but also fraught with peril. We're doing all sorts of wonky Win32 hacks to let the user drag that region as it is. I'd love to have the shadow work across that entire region, but we'll need to be careful to not break that behavior.
Author
Owner

@orcmid commented on GitHub (Jul 6, 2021):

@zadjii-msft > ... let the user drag that region

Yes, I have already become seriously frustrated by applications that shrink the draggable part of the title bar because of other cruft piling up there. (I confess that I have become fond of Edge's shrinkiable vertical tabs and how it leaves a wide title bar with the title of the tab for the current focus.)

@orcmid commented on GitHub (Jul 6, 2021): @zadjii-msft > ... let the user drag that region Yes, I have already become seriously frustrated by applications that shrink the draggable part of the title bar because of other cruft piling up there. (I confess that I have become fond of Edge's shrinkiable vertical tabs and how it leaves a wide title bar with the title of the tab for the current focus.)
Author
Owner

@DHowett commented on GitHub (Jul 6, 2021):

Yep -- if we do this, we'll need to make sure that AppHost can still communicate with NonClientIslandWindow to let it know where the draggable region is.

We used to do this because we had to punch a hole in the XAML window where no content could be displayed -- now it's just an implementation of convenience (read: software engineering is lazy and optimizes around local maxima sometimes 😁) since we have a different drag bar implementation. 😄

@DHowett commented on GitHub (Jul 6, 2021): Yep -- if we do this, we'll need to make sure that AppHost can still communicate with NonClientIslandWindow to let it know where the draggable region is. We used to do this because we had to punch a hole in the XAML window where no content could be displayed -- now it's just an implementation of convenience (read: software engineering is lazy and optimizes around local maxima sometimes :grin:) since we have a different drag bar implementation. :smile:
Author
Owner

@zadjii-msft commented on GitHub (Aug 23, 2023):

Oh hey we did this in #12916

@zadjii-msft commented on GitHub (Aug 23, 2023): Oh hey we did this in #12916
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#14096