mbnuqw / sidebery

Firefox extension for managing tabs and bookmarks in sidebar.
MIT License
3.26k stars 163 forks source link

Moving tabs across windows copies them instead #1723

Open sig-kill opened 1 month ago

sig-kill commented 1 month ago

Steps to reproduce

Actual behavior

Tab is copied over to the new window, and not active: image

Expected behavior

Tab should be moved from the old window and made active in the new window. Result in the example should be this: image

Moving tabs used to behave this way seamlessly (even moving a playing Youtube tab across windows worked without interrupting the video) so it seems like there's a regression.

System

Windows 10

Firefox version

129.0b5

Sidebery version

5.2.0

Logs

No console logs
ForsakenHarmony commented 1 month ago

Wonder if this is the same issue as #1714

geduardcatalindev commented 1 month ago

I have the exact same problem using the same versions of Firefox and Sidebery mentioned by @sig-kill

Bobo1239 commented 1 month ago

Did some debugging like in https://github.com/mbnuqw/sidebery/issues/1714#issuecomment-2267961026 and the regression is caused by the same Firefox change. Opened https://bugzilla.mozilla.org/show_bug.cgi?id=1912340 upstream since it's a different bug than what causes #1714 which already has a patch pending.

Bobo1239 commented 2 weeks ago

The just released Firefox Nightly contains the fix and afaict everything works again. :tada: All credits go to David Parks (handyman) who wrote the patches. Currently it is slated to land in Firefox 131 (scheduled for Oct. 1) but they may deem this important/unrisky enough to uplift to an earlier release.

Update: Got uplifted to beta (starting 130.0rc1) and should land on stable on September 3.

sig-kill commented 1 week ago

That's great news. Thanks for following up!