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

TST does not respect browser.tabs.warnOnCloseOtherTabs #1752

Closed
paninaro95 opened this issue Feb 1, 2018 · 9 comments
Closed

TST does not respect browser.tabs.warnOnCloseOtherTabs #1752

paninaro95 opened this issue Feb 1, 2018 · 9 comments
Labels

Comments

@paninaro95
Copy link

paninaro95 commented Feb 1, 2018

Short description

TST does not respect browser.tabs.warnOnCloseOtherTabs, i.e. TST does not show the message box that Firefox shows when you select "close other tabs" from the context menu of a tab.

By default, browser.tabs.warnOnCloseOtherTabs is set to true, so Firefox does warn you whereas TST does not. I just lost about 150 tabs as there is no undo possibility either 👎

Steps to reproduce

  1. Start Firefox with clean profile.
  2. Install TST.
  3. Open multiple tabs.
  4. Open the context menu of a tab inside the TST panel and select "close other tabs".
  5. All other tabs will be closed without a warning prompt.

Expected result

A warning prompt should be shown.

Actual result

No warning prompt is shown.

Environment

  • Platform (OS): Windows 10
  • Version of Firefox: 58.0.1
  • Version (or revision) of Tree Style Tab: 2.4.8
@Lej77
Copy link
Contributor

Lej77 commented Feb 1, 2018

If you haven't restarted (closed and then started) Firefox after losing your tabs it might be possible to restore your session to its state at last Firefox startup by closing Firefox and then moving and renaming:
FirefoxProfileFolder\sessionstore-backups\previous.jsonlz4
to
FirefoxProfileFolder\sessionstore.jsonlz4
and then starting Firefox again. You might want to backup previous.jsonlz4 before doing anything in case it goes wrong.

Firefox profiles folders are located in:
C:\Users\%UserName%\AppData\Roaming\Mozilla\Firefox\Profiles
and there should only be one folder in that directory if you haven't manually create new Firefox profiles.

I wrote about Firefox session management in issue 1678 so you could read more there if your interested.

@paninaro95
Copy link
Author

paninaro95 commented Feb 1, 2018

Thanks Lej77 for your hint -- I found that trick myself at https://support.mozilla.org/de/questions/1119692 although it is not up to date with the latest Firefox where the jsonlz4 files are used instead of js. I was lucky that I took a backup of my profile after I lost all my tabs, but still before I closed Firefox. I will now create a backup of my profile on a daily basis 😁

@lycsjm
Copy link
Contributor

lycsjm commented Feb 1, 2018

@paninaro95 Maybe you can try something like [Tab Session Manager for backup tabs. It can automatic backup periodically and works well with TST.

@piroor
Copy link
Owner

piroor commented Feb 5, 2018

Implemented by recent commits.

@piroor piroor added the fixed label Feb 5, 2018
@lycsjm
Copy link
Contributor

lycsjm commented Feb 10, 2018

I open 14 new tabs, auto grouping by TST(Firefox open warning dialog if I want to close window), and then collapse this tree then middle click to close tree, but no warning appear.

I am sure option in TST is clicked. What did i miss?

@piroor
Copy link
Owner

piroor commented Feb 10, 2018

@lycsjm Hmm, middle-click on a collapsed tree in the sidebar shows warning for me...

@lycsjm
Copy link
Contributor

lycsjm commented Feb 10, 2018

@piroor I left only one window, close others, and then toggle the option again, it works now.

@jonhermansen
Copy link

I'm not sure if this is related, but the "Close Other Tabs" context menu item doesn't always close all other tabs. It has been like this for a few weeks. Running 2.4.16

@piroor
Copy link
Owner

piroor commented May 1, 2019

I close this because outdated.

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

No branches or pull requests

5 participants