quill-image-uploader icon indicating copy to clipboard operation
quill-image-uploader copied to clipboard

Select multiple image files?

Open liuchao92 opened this issue 2 years ago • 1 comments

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch [email protected] for the project I'm working on.

I need to select multiple image files. I tried this solution. Maybe it's incomplete, but it worked.

Here is the diff that solved my problem:

diff --git a/node_modules/quill-image-uploader/src/quill.imageUploader.js b/node_modules/quill-image-uploader/src/quill.imageUploader.js
index 58fb063..fc72f38 100644
--- a/node_modules/quill-image-uploader/src/quill.imageUploader.js
+++ b/node_modules/quill-image-uploader/src/quill.imageUploader.js
@@ -29,6 +29,7 @@ class ImageUploader {
         this.fileHolder = document.createElement("input");
         this.fileHolder.setAttribute("type", "file");
         this.fileHolder.setAttribute("accept", "image/*");
+        this.fileHolder.setAttribute("multiple", "");
         this.fileHolder.setAttribute("style", "visibility:hidden");
 
         this.fileHolder.onchange = this.fileChanged.bind(this);
@@ -146,8 +147,9 @@ class ImageUploader {
     }
 
     fileChanged() {
-        const file = this.fileHolder.files[0];
-        this.readAndUploadFile(file);
+        for (const file of this.fileHolder.files) {
+            this.readAndUploadFile(file);
+        }
     }
 
     insertBase64Image(url) {

This issue body was partially generated by patch-package.

liuchao92 avatar Jan 06 '23 04:01 liuchao92

@NoelOConnell plz. 🙏

weolbu-owner avatar Jul 17 '23 05:07 weolbu-owner