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

[Dropzone] Inform the user of the number of files he uploads #2642

Open
wants to merge 1 commit into
base: 2.x
Choose a base branch
from

Conversation

rrr63
Copy link
Contributor

@rrr63 rrr63 commented Mar 19, 2025

Q A
Bug fix? no
New feature? yes
Docs? no
Issues #2567
License MIT

Hi, I was implementing a multiple file dropzone in one of my projects, and i was facing the same problem mentioned in #2567.
I was thinking for a solution, should we make a preview of all files if they're images ?
Sounds an interesting idea first, but if users upload a lot of images, it could be very ugly to manage the preview. Also, i was worried about avoiding regression for projects that are already in production and to break some custom dropzone styles.

The solution i used for my project is to show the number of extra files user uploads. I was inspired by the original file type field, it just shows us the number of files, according to the stylized dropzone of symfony ux, the first element (if it is an image) display a preview, and we write to user the number of extra files he uploads.

Here a screenshot of three fields :
-dropzone with single file
-dropzone with 3 files
-standard file type field with 3 files
Capture d’écran 2025-03-19 à 09 11 17

@carsonbot carsonbot added Dropzone Feature New Feature Status: Needs Review Needs to be reviewed labels Mar 19, 2025
Copy link

📊 Packages dist files size difference

Thanks for the PR! Here is the difference in size of the packages dist files between the base branch and the PR.
Please review the changes and make sure they are expected.

FileBefore (Size / Gzip)After (Size / Gzip)
Dropzone
controller.js 3.12 kB / 816 B 3.39 kB+8% 📈 / 913 B+12% 📈

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dropzone Feature New Feature Status: Needs Review Needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants