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

Allow never closing an entire tree when closing a collapsed tree #1474

Closed
Katrix opened this issue Oct 16, 2017 · 7 comments
Closed

Allow never closing an entire tree when closing a collapsed tree #1474

Katrix opened this issue Oct 16, 2017 · 7 comments

Comments

@Katrix
Copy link

Katrix commented Oct 16, 2017

Short description

Currently with TST if you close a collapsed tree, it will close all the tabs in the tree too. What I ask for is a config to disable this. I also suspect that it could be part of the reason for bugs like #1418, where you navigate away from a tab tree, close the parent tab, and because the tree is collapsed, the child tabs are also closed.

Steps to reproduce

  1. Open a few new tabs
  2. Organize them in a tree
  3. Collapse them
  4. Close the parent tab,

Expected result

The first child tab is promoted as parent

Actual result

All tabs are closed

Environment

  • Platform (OS): Windows 10 64-bit
  • Version of Firefox: 58.0a1 (2017-10-14) (64-bit)
  • Version (or revision) of Tree Style Tab: WE 2017-10-15
@piroor
Copy link
Owner

piroor commented Oct 17, 2017

Sorry I have no plan to change this behavior, because I believe it seems to be natural from their appearance.

@Katrix
Copy link
Author

Katrix commented Oct 17, 2017

I have to agree there. Would it be possible to make it a config, or make a tab closed outside a tree (no appearance which makes it seem natural) behave differently?

@nshtg
Copy link

nshtg commented Oct 17, 2017

Yet I agree with Katrix-, an option to do this would be nice.

@piroor
Copy link
Owner

piroor commented Oct 17, 2017

Because there are some mistakes, unexpectedly the config did not work as expected: "When a parent tab is moved or closed by someone from outside of the tree" => "Treat it as a parent only when the tree sidebar is visible". By recent commits, now the config should work as expected. When you choose this option, you'll see the behavior like you told when TST's sidebar is invisible.

If you mean that a new option to do same thing for "close tab by middle click on Firefox's native tab bar, even if the sidebar is visible", I have to do more changes.

@awused
Copy link

awused commented Dec 5, 2017

If you mean that a new option to do same thing for "close tab by middle click on Firefox's native tab bar, even if the sidebar is visible", I have to do more changes.

This is definitely the option I want. Right now it's too easy to close a large tree by mistake and end up unable to restore it.

@irvinm
Copy link
Contributor

irvinm commented May 12, 2019

@piroor please close this. (I assume you have not changes your mind on this one)

@piroor piroor removed the wontfix label May 12, 2019
piroor added a commit that referenced this issue May 12, 2019
@piroor
Copy link
Owner

piroor commented May 12, 2019

This was already implemented as the option "Treat as a parent tab by operations inside the tree sidebar. / Treat just as a solo tab, by operations outside the tree sidebar." and with recent changes context menu commands on the sidebar are also treated as "operations inside the tree sidebar".

@piroor piroor added the fixed label May 12, 2019
@piroor piroor closed this as completed May 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants