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

[Bug] 4.0.1 - Close multiple tabs warning dialog is empty #3474

Closed
Sada-Mio opened this issue Mar 12, 2024 · 10 comments
Closed

[Bug] 4.0.1 - Close multiple tabs warning dialog is empty #3474

Sada-Mio opened this issue Mar 12, 2024 · 10 comments

Comments

@Sada-Mio
Copy link

Abstract

Steps to reproduce

  1. Start Firefox with clean profile.
  2. Install TST.
  3. Open multiple tabs
  4. Select multiple tabs from sidebar to close

Expected result

  • Warning dialog popup with list of tabs

Actual result

  • Warning dialog popup is empty
  • No close tab or cancel button

Environment

  • Platform (OS): Win10
  • Version of Firefox: 123.0.1
  • Version (or revision) of Tree Style Tab: 4.0.1
@noc0lour
Copy link

I can confirm this behaviour on MacOS with Firefox 123.0.1 and Tree Style Tab 4.0.1

@markocelan
Copy link

Confirming this bug on Fedora Linux with Firefox 123.0.1 and Tree Style Tab 4.0.1

@piroor
Copy link
Owner

piroor commented Mar 12, 2024

Any error message in the error console opened by Ctrl-Shift-J, or the debugger for TST (see also https://github.com/piroor/treestyletab/wiki/How-to-inspect-tree-of-tabs#how-to-start-debugger-for-tree-style-tab )?

@NelsLucid
Copy link

I am also experiencing this bug on Ubuntu with Firefox 123.0.1 and Tree Style Tab 4.0.1. These are the errors I see in the debugger when I hit the "x" button on a tab and then see a blank dialog.

Uncaught (in promise) Error: Missing host permission for the tab undefined
managed storage is not provided [Configs.js:310:27](moz-extension://955503bc-0931-4434-9d54-130a06a6d978/extlib/Configs.js)
Uncaught TypeError: renderableTabs[firstRenderableIndex] is undefined
    renderVirtualScrollViewport moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js:251
    scrollTo moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js:508
    scrollStep moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js:638
    scrollStep moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js:643
    scrollStep moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js:643
    scrollStep moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js:643
    scrollStep moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js:643
    smoothScrollTo moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js:645
    smoothScrollTo moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js:614
    scrollTo moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js:492
    scrollToTab moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js:772
    async* moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/sidebar.js:822
    FrameRequestCallback* moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/sidebar.js:815
    EventListener.handleEvent* moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/sidebar.js:803
2 [scroll.js:251:5](moz-extension://955503bc-0931-4434-9d54-130a06a6d978/sidebar/scroll.js)

@piroor
Copy link
Owner

piroor commented Mar 12, 2024

Could you try again with granted "Access your data for all websites" permission? It may be a workaround.
image

Hmm, to avoid this problem, we need to introduce another problem again: Ctrl-Shift-T to restore close tabs will "restores" a blank window corresponding to the dialog.

@NelsLucid
Copy link

Yes, that seems to work. Thank you for the workaround.

@piroor
Copy link
Owner

piroor commented Mar 12, 2024

I've released 4.0.2 with a fix for this. Please wait until it is published.

@piroor piroor added the fixed label Mar 12, 2024
@ocolluphid
Copy link

thanks, this also worked for me! should we revert to old settings when 4.0.2 is released?

@piroor
Copy link
Owner

piroor commented Mar 13, 2024

The permission is also required if you hope the keyboard shortcut Ctrl-Tab/Ctrl-Shift-Tab to switch tabs skipping collapsed tabs. Otherwise you can remove the permission.

@markocelan
Copy link

After the upgrade to 4.0.2(without updating any permissions) it started to work as expected. Thank you!

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

6 participants