search

ajaxorg / ace

Ace (Ajax.org Cloud9 Editor)
https://ace.c9.io
Other
26.67k stars 5.28k forks source link

e.dataTransfer is null #5622

Open lavanyaburlagadda1807 opened 1 month ago

lavanyaburlagadda1807 commented 1 month ago

Describe the bug

After writing some code in ace editor, when we are dragging the the text that is written, it properly being dragged but along with that we are getting e.dataTransfer is null and reporting to sentry.

We are getting this error only in firefox

Expected Behavior

Error should not be caused

Current Behavior

Getting error as e.dataTransfer is null

Reproduction Steps

Even in ace editor as well we are getting this issue.

https://mkslanc.github.io/ace-playground/#hello-world

Write some code, select it and drag it

Possible Solution

No response

Additional Information/Context

No response

Ace Version / Browser / OS / Keyboard layout

1.33.0 / Firefox / Ubuntu

andredcoliveira commented 1 month ago

I'm not familiar with this feature. How do you drag the code? If I select code, and then try to drag it, it simply starts a new selection. Even if I set draggable=true in the HTML element that has Ace.

Can you provide a fully reproducible code snippet?

In any case, the error is probably coming from this place: https://github.com/ajaxorg/ace/blob/0192880b587797eeb65c43b3a6d882c0219b19e2/src/mouse/dragdrop_handler.js#L106

lavanyaburlagadda1807 commented 1 month ago

Could you please check the below video, we are just using Ace editor, we haven't written anything related to drag

Screencast from 21-08-24 11:56:03 AM IST.webm I am using firefox version 129.0 (64-bit)