Skip to content

Conversation

@karheda
Copy link

@karheda karheda commented Dec 1, 2025

Q A
Bug fix? no
New feature? yes
Deprecations? no
Documentation? no
Issues Fix #2567
License MIT

@karheda karheda requested a review from Kocal as a code owner December 1, 2025 10:39
@carsonbot carsonbot added Bug Bug Fix Deprecation Documentation Improvements or additions to documentation Feature New Feature Status: Needs Review Needs to be reviewed labels Dec 1, 2025
@carsonbot carsonbot changed the title Dropzone multiple uploads Dropzone multiple uploads Dec 1, 2025
@karheda karheda changed the title Dropzone multiple uploads [Dropzone] Dropzone multiple uploads Dec 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2025

📊 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.d.ts 723 B / 337 B 958 B+33% 📈 / 425 B+26% 📈
controller.js 2.92 kB / 819 B 4.43 kB+52% 📈 / 1.34 kB+67% 📈
style.min.css 944 B / 468 B 959 B+2% 📈 / 472 B+1% 📈

@karheda
Copy link
Author

karheda commented Dec 1, 2025

Hi @Kocal I had to create new PR, because I couldn't change my default branch from 2.x to dropzone-multiple-uploads branch in the previews PR. We can close the preview PR. Now we have to solve 3 failing checks of PHP too. Could you tell me if I have made the changes well?
Thank you!

Carlos Hernández added 16 commits December 1, 2025 12:11
- Format array declarations for better readability in `controller.ts`.
- Correct typos in `CHANGELOG.md`.
- Adjust imports for consistency in `vitest.config.mjs`.
- Refactor `vitest.setup.js` for improved code structure.
…d UI logic for preview visibility and structure.
…coverage, and adjust styles.

# Conflicts:
#	src/Dropzone/assets/dist/controller.js
#	src/Dropzone/assets/src/controller.ts
#	src/Dropzone/assets/test/unit/controller.test.ts
#	src/Dropzone/assets/test/vitest.setup.js
#	src/Dropzone/assets/vitest.config.mjs
- Format array declarations for better readability in `controller.ts`.
- Correct typos in `CHANGELOG.md`.
- Adjust imports for consistency in `vitest.config.mjs`.
- Refactor `vitest.setup.js` for improved code structure.

# Conflicts:
#	src/Dropzone/CHANGELOG.md
#	src/Dropzone/assets/src/controller.ts
… files.

# Conflicts:
#	src/Dropzone/assets/src/controller.ts
…d UI logic for preview visibility and structure.

# Conflicts:
#	src/Dropzone/assets/test/unit/controller.test.ts
#	src/Dropzone/templates/form_theme.html.twig
- Format array declarations for better readability in `controller.ts`.
- Correct typos in `CHANGELOG.md`.
- Adjust imports for consistency in `vitest.config.mjs`.
- Refactor `vitest.setup.js` for improved code structure.
@karheda karheda force-pushed the dropzone-multiple-uploads branch from a204ffc to d6bf015 Compare December 1, 2025 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Bug Fix Deprecation Documentation Improvements or additions to documentation Feature New Feature Status: Needs Review Needs to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Dropzone] Multiple uploads only show single file and no possibility to add more files

2 participants