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

Dragging already added file and dropping it, duplicates file #5622

Closed
2 tasks done
gongAll opened this issue Jan 23, 2025 · 3 comments · Fixed by #5626
Closed
2 tasks done

Dragging already added file and dropping it, duplicates file #5622

gongAll opened this issue Jan 23, 2025 · 3 comments · Fixed by #5626
Labels

Comments

@gongAll
Copy link

gongAll commented Jan 23, 2025

Initial checklist

  • I understand this is a bug report and questions should be posted in the Community Forum
  • I searched issues and couldn’t find anything (or linked relevant results below)

Link to runnable example

https://uppy.io/examples/

Steps to reproduce

  • Add .png file
  • After being added, drag file around
  • Drop file when prompted to do so ("Drop files here" appears)

Expected behavior

Should not allow to drop file. Perhaps should allow to reorder files instead, when files have already been added.

Actual behavior

Result: File is duplicated how many times I do it (literally to infinity)

@gongAll gongAll added the Bug label Jan 23, 2025
@Murderlon
Copy link
Member

Not sure why but I can't reproduce this locally for the website nor in another dev setup 🤔

However I did found a bug where the Url plugin tries to fetch the link when dropping the same file.

@gongAll
Copy link
Author

gongAll commented Jan 27, 2025

Did create a streamable on your example website.

@gongAll
Copy link
Author

gongAll commented Feb 3, 2025

@Murderlon

Why did you close this?

From what I understood, you only fixed the Companion warning, correct? The original bug is still present from what I tested just now.

I put a streamable because the reproduction is literally in your example page. I don't believe I can do much more on my end.

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

Successfully merging a pull request may close this issue.

2 participants