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] Cannot drag-and-drop outside of sidebar to open new window (FF version >= v129.0b2) #1714

Closed
the-nelsonator opened this issue Jul 12, 2024 · 13 comments

Comments

@the-nelsonator
Copy link

the-nelsonator commented Jul 12, 2024

Steps to reproduce

  • Open Firefox Dev Edition 129.0b2, or Firefox Nightly v130.0a1 (2024-07-12) in a new profile
  • Install Sidebery v5.2.0
  • Try to drag a tab outside of sidebar

Actual behavior

Does nothing

sideberydraganddrop

Expected behavior

Opens a new window with the tab

System

Windows 11

Firefox version

Open Firefox Dev Edition 129.0b2, or Firefox Nightly v130.0a1 (2024-07-12)

Sidebery version

v5.2.0

Logs

No logs at all in console.

Notes:

  • Works as expected on vanilla Firefox 128.0
  • Swapping the "Action on drag-and-drop to the outside of sidebar" setting to "Pass dragged data to the target" shows that it's still registering the drag and drop, i.e. if I drag a tab in here: moz-extension://b6832724-3975-40ee-953f-845199cf5989/page.setup/setup.html#settings_dnd
@the-nelsonator the-nelsonator changed the title [BUG] On Dev Edition and Nightly, cannot drag-and-drop outside of sidebar to open new window [BUG] Cannot drag-and-drop outside of sidebar to open new window (On Dev Edition and Nightly) Jul 12, 2024
@the-nelsonator the-nelsonator changed the title [BUG] Cannot drag-and-drop outside of sidebar to open new window (On Dev Edition and Nightly) [BUG] Cannot drag-and-drop outside of sidebar to open new window (FF version >= v129.0b2) Jul 12, 2024
@the-nelsonator
Copy link
Author

@rebelonion I noticed you used Dev Edition on 06/26 to demo your PR related to drag and drop: #1698.

On my machine drag and drop is not working as of Dev Edition 129.0b2 (released 7/10). Is the same true for you? Helps narrow down timeframe or if this is just due to something in my environment

@sebibasti0815
Copy link

Problem exists in 129.0b3, too. "Workaround" using ff extension tabdetach and using keyboard shortcuts.

@the-nelsonator
Copy link
Author

the-nelsonator commented Jul 15, 2024

Right-click tab -> Reopen in -> New window is also a less convenient workaround

@sebibasti0815
Copy link

"Move to", but, yeah 👌😂

@beracira
Copy link

Regarding the note: I can reproduce the issue on

Version 	128.0
Build ID 	20240704121409

The ability to rearrange tabs with the sidebar is also affected.

@nirurin
Copy link

nirurin commented Jul 18, 2024

For me the issue (on 128) is that when I try and open a tab in a new window by dragging (or using the 'open in new window' right click menu option) the window opens fine... but sidebery is no longer open in that window. Not sure if that's related or a different bug entirely.

@beracira
Copy link

beracira commented Jul 20, 2024

I found a fix (well, as least for my issue):
Changing dom.events.dataTransfer.imageAsFile.enabled to true in about:config seems to fix the issue.

It looks like a regression: bugzilla link.

@nirurin
Copy link

nirurin commented Jul 20, 2024

Ah unfortunately that doesn't fix my issue. Even more unfortunately i haven't seen the dev respond to any bug reports for a few months.

@sebibasti0815
Copy link

sebibasti0815 commented Jul 20, 2024

I found a fix (well, as least for my issue): Changing dom.events.dataTransfer.imageAsFile.enabled to true in about:config seems to fix the issue.

It looks like a regression: bugzilla link.

That doesn't fix the problem here. It's a different problem, dragging images - our problem in this case is dragging tabs.

@the-nelsonator
Copy link
Author

Yeah, tried dom.events.dataTransfer.imageAsFile.enabled to true on v129.0b6 and can confirm did not fix the drag'n'drop to open in new window

@Bobo1239
Copy link

Bobo1239 commented Aug 5, 2024

I've bisected the issue to this Firefox change which makes sense since it touches the drag'n drop subsystem. The effect is that in this Sidebery line e.dataTransfer is now null instead of a DataTransfer object. (which alters our control flow) According to the MDN docs this should never happen when the event is dispatched by the browser which is the case here so I'd say this is a Firefox bug.

Unfortunately I don't have time to dive more deeply into the Firefox change so somebody else will have to do that... (I've also open a bugzilla issue for this: https://bugzilla.mozilla.org/show_bug.cgi?id=1911486)

@the-nelsonator
Copy link
Author

the-nelsonator commented Aug 14, 2024

@Bobo1239 Thank for identifying the issue (noting down that bisection tool, very cool) and reporting on Bugzilla. Looks like David Parks/handyman already got a fix in - drag and drop to open in other window works as expected in v131! (Only on Nightly for now)

Looks like, as expected since you opened a separate Bugzilla, this did not fix the issue with tabs "cloning" in the other window instead of moving.

@sebibasti0815
Copy link

drag and drop to open in other window works as expected in v131! (Only on Nightly for now)

It works in 130.0b4, too. But the other issue (tabs cloning instead of moving) is there, too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants