Remove workaround "When we add a new tab in compact mode, re-enforce Compact mode" #9280

Closed
opened 2026-01-31 01:50:35 +00:00 by claunia · 8 comments
Owner

Originally created by @zadjii-msft on GitHub (Jun 25, 2020).

Originally assigned to: @marcelwgn on GitHub.

When we get a version of WinUI with a fix for https://github.com/microsoft/microsoft-ui-xaml/issues/2711, we can go ahead and revert #6670. This task is tracking that future work.

Originally created by @zadjii-msft on GitHub (Jun 25, 2020). Originally assigned to: @marcelwgn on GitHub. When we get a version of WinUI with a fix for https://github.com/microsoft/microsoft-ui-xaml/issues/2711, we can go ahead and revert #6670. This task is tracking that future work.
Author
Owner

@DHowett commented on GitHub (Jun 25, 2020):

Triaged

@DHowett commented on GitHub (Jun 25, 2020): Triaged
Author
Owner

@marcelwgn commented on GitHub (Jul 10, 2020):

The new prerelease of WinUI (2.5.0.prerelease.200708003) contains the fix for TabView, so with that prerelease, the workaround can be removed.

@marcelwgn commented on GitHub (Jul 10, 2020): The new prerelease of WinUI (2.5.0.prerelease.200708003) contains the fix for TabView, so with that prerelease, the workaround can be removed.
Author
Owner

@marcelwgn commented on GitHub (Aug 28, 2020):

I would like to work on this issue.

@marcelwgn commented on GitHub (Aug 28, 2020): I would like to work on this issue.
Author
Owner

@zadjii-msft commented on GitHub (Aug 28, 2020):

Go right ahead! You'll probably need to bump our MUX version up across the entire solution. I'd take a look at https://github.com/microsoft/terminal/blob/master/doc/building.md#updating-nuget-package-references for a quick and dirty script for doing that part ☺️

@zadjii-msft commented on GitHub (Aug 28, 2020): Go right ahead! You'll probably need to bump our MUX version up across the entire solution. I'd take a look at https://github.com/microsoft/terminal/blob/master/doc/building.md#updating-nuget-package-references for a quick and dirty script for doing that part ☺️
Author
Owner

@marcelwgn commented on GitHub (Aug 28, 2020):

Thanks for the link, I just noticed that for some reason, in the documentation, I chose to make the Oldversion higher then the Newversion. Mind if I create a separate PR to fix this? The current way it is presented might be a bit confusing for other folks reading that part.

@marcelwgn commented on GitHub (Aug 28, 2020): Thanks for the link, I just noticed that for some reason, in the documentation, I chose to make the Oldversion higher then the Newversion. Mind if I create a separate PR to fix this? The current way it is presented might be a bit confusing for other folks reading that part.
Author
Owner

@zadjii-msft commented on GitHub (Aug 28, 2020):

Oh yea totally - that's probably my bad from the last time I used this script. It probably also bumped the version number within the script itself 😃

@zadjii-msft commented on GitHub (Aug 28, 2020): Oh yea totally - that's probably my bad from the last time I used this script. It probably also bumped the version number _within the script itself_ 😃
Author
Owner

@marcelwgn commented on GitHub (Aug 28, 2020):

Oh right haha, just noticed it doing this on my machine too! Maybe we should choose some nonexistent versions so this doesn't happen again 🤔?

@marcelwgn commented on GitHub (Aug 28, 2020): Oh right haha, just noticed it doing this on my machine too! Maybe we should choose some nonexistent versions so this doesn't happen again 🤔?
Author
Owner

@zadjii-msft commented on GitHub (Aug 28, 2020):

Yea probably haha

@zadjii-msft commented on GitHub (Aug 28, 2020): Yea probably haha
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#9280