Close an unfocused tab (using mouse) results in focus switched #1113

Closed
opened 2026-01-30 22:16:46 +00:00 by claunia · 8 comments
Owner

Originally created by @bef178 on GitHub (May 15, 2019).

This bug comes with #737. Very sorry for not having a full test.

Repo:
open tab a,b,c,d
activate c
mouse close a
expected: c is shown and is the only highlight in tabview
actual: b is shown as focused tab

Originally created by @bef178 on GitHub (May 15, 2019). This bug comes with #737. Very sorry for not having a full test. Repo: open tab a,b,c,d activate c mouse close a expected: c is shown and is the only highlight in tabview actual: b is shown as focused tab
claunia added the Issue-BugNeeds-Tag-FixProduct-TerminalArea-UserInterface labels 2026-01-30 22:16:46 +00:00
Author
Owner

@bef178 commented on GitHub (May 15, 2019):

Manual test cases:

open tab a,b,c,d
activate a
ctrl-w
ctrl-w
expect c is shown and is the only highlight in tabview

open tab a,b,c,d
activate a
mouse close a
mouse close b
expect c is shown and is the only highlight in tabview

open tab a,b,c,d
activate c
mouse close a
expect c is shown and is the only highlight in tabview
alt-1
expect b is shown and is the only highlight in tabview

open tab a,b,c,d
activate c
mouse close a
expect c is shown and is the only highlight in tabview
mouse activate b
expect b is shown and is the only highlight in tabview

open tab a,b,c,d,e
activate c
ctrl-w
ctrl-w
ctrl-w
expect d is shown and is the only highlight in tabview
ctrl-w
expect e is shown and tabview is hided

@bef178 commented on GitHub (May 15, 2019): Manual test cases: open tab a,b,c,d activate a ctrl-w ctrl-w expect c is shown and is the only highlight in tabview open tab a,b,c,d activate a mouse close a mouse close b expect c is shown and is the only highlight in tabview open tab a,b,c,d activate c mouse close a expect c is shown and is the only highlight in tabview alt-1 expect b is shown and is the only highlight in tabview open tab a,b,c,d activate c mouse close a expect c is shown and is the only highlight in tabview mouse activate b expect b is shown and is the only highlight in tabview open tab a,b,c,d,e activate c ctrl-w ctrl-w ctrl-w expect d is shown and is the only highlight in tabview ctrl-w expect e is shown and tabview is hided
Author
Owner

@bef178 commented on GitHub (May 15, 2019):

And is there any place to put manual test cases? In case of unconsciously / accidentally breaking other features.

@bef178 commented on GitHub (May 15, 2019): And is there any place to put manual test cases? In case of unconsciously / accidentally breaking other features.
Author
Owner

@zadjii-msft commented on GitHub (May 15, 2019):

We actually don't have any UI tests set up quite yet, though I think someone is working on them, so you don't really need to worry about adding them yourself quite yet.

@zadjii-msft commented on GitHub (May 15, 2019): We actually don't have any UI tests set up quite yet, though I think someone is working on them, so you don't really need to worry about adding them yourself quite yet.
Author
Owner

@bef178 commented on GitHub (May 15, 2019):

I wish I will not break other features.

@bef178 commented on GitHub (May 15, 2019): I wish I will not break other features.
Author
Owner

@binarycrusader commented on GitHub (May 16, 2019):

I wish I will not break other features.

Any work worth doing always has a chance of bugs--the important thing is to record them when found, learn the cause, and work on getting them fixed 🙌

@binarycrusader commented on GitHub (May 16, 2019): > I wish I will not break other features. Any work worth doing always has a chance of bugs--the important thing is to record them when found, learn the cause, and work on getting them fixed 🙌
Author
Owner

@bef178 commented on GitHub (May 16, 2019):

You're right

@bef178 commented on GitHub (May 16, 2019): You're right
Author
Owner

@bef178 commented on GitHub (May 23, 2019):

The described behavior should be gone in master. Closing.

@bef178 commented on GitHub (May 23, 2019): The described behavior should be gone in master. Closing.
Author
Owner

@zeimao77 commented on GitHub (May 30, 2019):

linux vim ctrl-w
Switching the VIM window causes the terminal to close

@zeimao77 commented on GitHub (May 30, 2019): linux vim ctrl-w Switching the VIM window causes the terminal to close
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#1113