Skip to content
This repository has been archived by the owner on Jul 26, 2023. It is now read-only.

Fix drag&drop #24

Open
NoobTracker opened this issue Aug 22, 2020 · 16 comments
Open

Fix drag&drop #24

NoobTracker opened this issue Aug 22, 2020 · 16 comments

Comments

@NoobTracker
Copy link

A problem I have is that drag&drop isn't working. I just get that "You can't drop files here" symbol thingy when I wanna do exactly that. I'm using Firefox.

@BoomerScratch
Copy link
Contributor

I'm on Windows 10 Chrome, and it works fine.

@NoobTracker
Copy link
Author

@BoomerScratch Can you try it with the userscript instead of the Chrome thingy?

@BoomerScratch
Copy link
Contributor

@NoobTracker I already use the userscript.

@jeffalo
Copy link
Owner

jeffalo commented Aug 22, 2020

firefox doesn't support the web api thats used for drag and drop i think

@jeffalo
Copy link
Owner

jeffalo commented Aug 22, 2020

are there any errors @NoobTracker ?

@NoobTracker
Copy link
Author

@JeffaloBob Errors, you mean when I open the console? Let me check that ... (btw, I'm sure that Firefox supports the drag&drop event.preventDefault(); thing, I messed with that stuff a year ago)

@NoobTracker
Copy link
Author

@BoomerScratch Ah.

@jeffalo
Copy link
Owner

jeffalo commented Aug 23, 2020

yes console errors

@NoobTracker
Copy link
Author

I don't think so ...

@NoobTracker
Copy link
Author

Nope.

@NoobTracker
Copy link
Author

@JeffaloBob No errors, and also no errors when I try to drop something.

@NoobTracker
Copy link
Author

Wait, don't you also need preventDefault when you hover over the input?

@NoobTracker
Copy link
Author

@JeffaloBob I think that the lack of preventDefault is the reason and Chrome may handle that differently but currently my scratch accounts are blocked so I can't check it.
(jeff, it would be great if you could enable the chat, I just get a 522)

@NoobTracker
Copy link
Author

@JeffaloBob Bump?

@jeffalo jeffalo closed this as completed Aug 25, 2020
@jeffalo jeffalo reopened this Aug 25, 2020
@NoobTracker
Copy link
Author

What was that?

@NoobTracker
Copy link
Author

So, should I just make a PR with ondragover?

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

No branches or pull requests

3 participants