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

tab selected should be adjacent when closing last one. #13912

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

rschnekenbu
Copy link
Contributor

What it does

In the case the last tab is closed in a tab bar, the tab selected should be the last one instead of the first one, in case there is no focus history.

fixes #13886

contributed on behalf of STMicroelectronics

How to test

Reproduce steps in the video in #13886:

  1. Open several editors (6 or more), select one in the middle, close it,
  2. Close some editors around,
  3. When 3 or more are still opened, close the last one in the bar.
  4. The selected tab should be the new last one, and not the first one on the bar.

Follow-ups

no known follow-ups

Review checklist

Reminder for reviewers

Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍

fixes eclipse-theia#13886

contributed on behalf of STMicroelectronics

Signed-off-by: Remi Schnekenburger <rschnekenburger@eclipsesource.com>
@rschnekenbu rschnekenbu merged commit a4bf86f into eclipse-theia:master Jul 18, 2024
14 checks passed
@rschnekenbu rschnekenbu deleted the issues/13886 branch July 18, 2024 07:57
@sgraband sgraband added this to the 1.52.0 milestone Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Focused tab is not the expected one when closing the last in the tabbar
3 participants