Editor: drop event in UIOutliner of ui.three.js not being call

[whyqiliao]

Description

Edge 124.0.2478.51 x64

function onDrop(event) not being call when darg and drop things in UIOutliner

Solution

add event.preventDefault(); in onDragOver()

Alternatives

Version
r153

Additional context

No response

[Mugen87]

function onDrop(event) not being call when darg and drop things in UIOutliner

I can't reproduce. The event listener is executed, I have verified this by setting a break point. It's also not clear to me how a call of preventDefault() in onDragOver() should fix an issue in onDrop().

Do you mind explaining what kind of user interaction is currently broken in the editor? Please share step-by-step instructions or a video so it's clear for us how to reproduce the issue.

[whyqiliao]

no problem.
First of all ,i modify the editor layout a liitle bit, but didnt change the core things.
Here is the capture video

bandicam.2024-04-25.17-19-41-065.mp4

[whyqiliao]

it also occur that the browser wiredly open a new tab searching 'foo'
i found the 'foo' in Firefox needs solution in onDragStart

[Mugen87]

it also occur that the browser wiredly open a new tab searching 'foo'

Interesting! I've tested with Chrome, Firefox and Safari but I'm unable to reproduce the issue.

Does the tab issue only happen in Edge or it other browser as well?

[whyqiliao]

I didnt try other browser, maybe it just occur in Edge?
Here is the edge info

[whyqiliao]

It also happen in Chrome 88 x64 (i use a clean version)

bandicam.2024-04-25.17-46-08-362.mp4

[Mugen87]

i modify the editor layout a liitle bit,

Is it possible to reproduce the issue with the unmodified version of the editor? Meaning: https://threejs.org/editor/

[whyqiliao]

the unmodified version works well, i should check my code

[Mugen87]

As long as the error does not occur in the official (unmodified) version of the editor, there is no need for a patch in this repository.