[Bug] Move to New Window button causes all tab bars to reload twice after the new window opens

[hhyyrylainen]

[Bug] Move to New Window button causes all tab bars to reload twice after the new window opens

[AKruimink]

Not sure if it's related at all to this.
Im currently running 4.0.13 (and Firefox 124.0.2 (64-bit) on Windows), and cannot get tabs to open in new windows what so ever.

When using the 'Move To New Window' nothing happens (it's not greyed out).
And when dragging the tab outside of TTS, instead of opening in a new window, it just display a blocked mouse pointer (circle with stripe) and the tab just jumps back to TTS.

I tried using Firefox it's native tab's and they work just fine.
Already reinstalled TTS as well, which doesnt seem to fix it either.
I changed the Tree behaviour to what is shown in the image above, this to didn't change anything.

[piroor]

@bennettscience Thanks, I missed fullscreen or maximized window cases. The commit 05ea7cb should fix the error.

[piroor]

Hmm, it is strange... Anyway I've introduced a change b05acbc to enforce the "Move to New Window" command to move whole tree.

[piroor]

The tab moves to another window (but its child tabs don't, this seems to be also a new bug / changed behaviour in the latest version)

This result is from TST options => "Tree Behavior" => "When a parent tab is Closed or Moved" => "Recommended preset using the browser's native tab bar as the Solo Tab Operation UI". "Recommended preset using Consistent Tab Behaviors controlled under TST" will move descendant tabs together on this case.

[hhyyrylainen]

If I understood your message correctly, this is the setting I should use:

I was already using that setting. I tried the other mode as well but it did the same thing (child tabs didn't move).

[piroor]

After more research I've realized that the sidebar panel can be reloaded accidentally by a listener left registered unintentionally. The commit e1a574b should reduce such a needless reloading of the sidebar. Could you try the latest development build of TST?
Instructions: https://github.com/piroor/treestyletab?tab=readme-ov-file#development-builds

[hhyyrylainen]

I downloaded the test build and got it working in a clean firefox profile that didn't have TST installed already. I was able to test that the two problems I noticed are gone with that build. I hope that when made into a proper release it all works fine as currently when I tried to install that update to a firefox profile that already had TST installed that caused the move to new window option to stop entirely working. But that just might be an effect of loading it as a temporary plugin update.
Thanks for the very quick responses.

[hhyyrylainen]

Not sure if it's related at all to this.
Im currently running 4.0.13 (and Firefox 124.0.2 (64-bit) on Windows), and cannot get tabs to open in new windows what so ever.

I just noticed that that started happening to me yesterday as well. I hadn't restarted my computer or firefox when that new problem started happening.

[bennettscience]

I noticed the same behavior on Windows. Here's the console output when trying to move a tab to a new window:

Uncaught (in promise) Error: Type error for parameter createData (Error processing left: Expected integer instead of null) for windows.create.
    openNewWindowFromTabs moz-extension://cb9b7a44-0223-4733-a17e-7bd12231a72b/background/tree.js:1669
    openTabsInWindow moz-extension://cb9b7a44-0223-4733-a17e-7bd12231a72b/background/commands.js:958
    openTabInWindow moz-extension://cb9b7a44-0223-4733-a17e-7bd12231a72b/background/commands.js:940
    onClick moz-extension://cb9b7a44-0223-4733-a17e-7bd12231a72b/background/tab-context-menu.js:1223
[tree.js:1669:45](moz-extension://cb9b7a44-0223-4733-a17e-7bd12231a72b/background/tree.js)

​

[github-actions]

This issue has been closed due to no response within 14 days after labeled as "maybe fixed", 7 days after last reopened, and 7 days after last commented.