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

Two CRASHs : Close Tab #252

Closed
peter8777555 opened this issue Jul 6, 2020 · 0 comments
Closed

Two CRASHs : Close Tab #252

peter8777555 opened this issue Jul 6, 2020 · 0 comments

Comments

@peter8777555
Copy link

peter8777555 commented Jul 6, 2020

Two crashs below:

Crash 1 :
Only one Tab and close Tab.

Crash 2 :
First Tab and Ctrl+E 10 Tabs
and Close FIRST Tab

PS:
I use Tool Button -> "Close the current tab" to close tab.
But if i use Ctrl+W,it is OK.
What happen ?

@derceg derceg closed this as completed in 575e769 Oct 13, 2020
derceg added a commit that referenced this issue Oct 31, 2020
When the selected tab is closed and there is no previously selected tab
(i.e. because only a single tab has ever been selected), the selection
is transferred to the next tab. The only exception to this is when the
last tab is closed, in which case the selection is transferred to the
previous tab.

Previously, the selection wasn't been set properly in the usual case. If
the last tab was closed, the selection would be correctly set to the
previous tab. However, if any tab but the last was closed, the selection
would be kept as-is and the tab would be closed.

This would then result in a situation in which no tab was selected,
which would cause the application to crash, due to the assumption that
there's always a selected tab being broken.

To fix this, the next tab is now selected before a tab (not in the last
position) is closed.

This fixes #252.

(cherry picked from commit 575e769)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant