Close Menu, "Close other tabs" and "Close tabs to right" always operate on tab index 0, are always enabled #11069

Closed
opened 2026-01-31 02:37:48 +00:00 by claunia · 5 comments
Owner

Originally created by @DHowett on GitHub (Oct 20, 2020).

Originally assigned to: @DHowett, @carlos-zamora on GitHub.

I'm not sure if this is an issue in main or an issue in feature/settings-ui.

/cc @mpela81

Originally created by @DHowett on GitHub (Oct 20, 2020). Originally assigned to: @DHowett, @carlos-zamora on GitHub. I'm not sure if this is an issue in main or an issue in feature/settings-ui. /cc @mpela81
Author
Owner

@mpela81 commented on GitHub (Oct 20, 2020):

Looks fine on master, maybe it wasn't fully merged onto the feature branch?

immagine

@mpela81 commented on GitHub (Oct 20, 2020): Looks fine on master, maybe it wasn't fully merged onto the feature branch? ![immagine](https://user-images.githubusercontent.com/1140981/96576414-7331ae80-12d2-11eb-9880-fdbb78ce0ee1.png)
Author
Owner

@DHowett commented on GitHub (Oct 21, 2020):

Excellent. In that case, it's on me 😄

@DHowett commented on GitHub (Oct 21, 2020): Excellent. In that case, it's on me :smile:
Author
Owner

@zadjii-msft commented on GitHub (Oct 21, 2020):

Uh oh, I'm seeing this on the 1.5.2941.0 selfhost the team is working with as well.

@zadjii-msft commented on GitHub (Oct 21, 2020): Uh oh, I'm seeing this on the 1.5.2941.0 selfhost the team is working with as well.
Author
Owner

@DHowett commented on GitHub (Oct 21, 2020):

Fortunately, that is built off feature/settings-ui!

@DHowett commented on GitHub (Oct 21, 2020): Fortunately, that is built off feature/settings-ui!
Author
Owner

@DHowett commented on GitHub (Oct 22, 2020):

Looks like this was @carlos-zamora's merge commit

commit 0ce05b5405df96a829d4756f44e4b1daffe28083
Merge: e00275232 743283e43
Author: Carlos Zamora <cazamor@microsoft.com>
Date:   Fri Oct 16 12:48:21 2020 -0700

    Merge branch 'master' into feature/settings-ui

Carlos, this commit lost the call to _EnableCloseMenuItems in the various tab implementations that controlled whether the menu items were enabled/disabled.

@DHowett commented on GitHub (Oct 22, 2020): Looks like this was @carlos-zamora's merge commit ``` commit 0ce05b5405df96a829d4756f44e4b1daffe28083 Merge: e00275232 743283e43 Author: Carlos Zamora <cazamor@microsoft.com> Date: Fri Oct 16 12:48:21 2020 -0700 Merge branch 'master' into feature/settings-ui ``` Carlos, this commit lost the call to `_EnableCloseMenuItems` in the various tab implementations that controlled whether the menu items were enabled/disabled.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#11069