Skip to content

Add a "Close other tabs" button to the tabs panel - #2055

Merged
svillar merged 1 commit into
Igalia:mainfrom
monofunc:feature/close-other-tabs
Sep 1, 2026
Merged

Add a "Close other tabs" button to the tabs panel#2055
svillar merged 1 commit into
Igalia:mainfrom
monofunc:feature/close-other-tabs

Conversation

@monofunc

@monofunc monofunc commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

The button sits in the selection bar next to "Close all" and closes every tab that is not currently shown in a window. WebExtension sessions are never closed. The button is hidden when there is nothing to close.

#2010 (comment)

@svillar svillar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the patch!

mUnselectTabs.setVisibility(View.GONE);
mCloseTabsAllButton.setVisibility(View.VISIBLE);
mCloseOtherTabsButton.setVisibility(View.VISIBLE);
mCloseOtherTabsButton.setEnabled(!SessionStore.get().getInactiveSessions(mPrivateMode).isEmpty());

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's do the same as the others and change visibility instead of disabling

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Should it be shown whenever the other buttons are shown or hidden when there's nothing to close?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The logic to show it was fine (checking inactive sessions), the only thing to change is that the condition should change the visibility status not whether it's enabled/disabled

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ok, updated.

The button sits in the selection bar next to "Close all" and closes
every tab that is not currently shown in a window. WebExtension
sessions are never closed. The button is hidden when there is
nothing to close.
@monofunc
monofunc force-pushed the feature/close-other-tabs branch from 0a08cab to ea1ea34 Compare August 21, 2026 13:17
@monofunc
monofunc requested a review from svillar August 21, 2026 13:19

@svillar svillar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@svillar svillar added the release_candidate PR that should be part of the next release label Sep 1, 2026
@svillar
svillar merged commit dc38a5e into Igalia:main Sep 1, 2026
6 checks passed
@svillar svillar added this to the v1.9.1 milestone Sep 1, 2026
@svillar svillar removed the release_candidate PR that should be part of the next release label Sep 1, 2026
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

Successfully merging this pull request may close these issues.

2 participants