kinow commented on PR #2123:
URL: https://github.com/apache/jena/pull/2123#issuecomment-2019033851

   Refactored to match what we discussed on Twitter, @neumarcx. Edited commit, 
and did manual tests. It's now missing just the unit/e2e tests, and then it'll 
be ready to go. Some screenshots of what it looks like in action.
   
   Blank page when you open the page to add data:
   
   
![image](https://github.com/apache/jena/assets/304786/b22d418f-5384-4efb-a684-9c6612fb21f0)
   
   Selecting one invalid file, and two valid files.
   
   
![image](https://github.com/apache/jena/assets/304786/82e400d2-aa8e-480d-9bf0-9aa63a6bd8af)
   
   Hitting upload all:
   
   
![image](https://github.com/apache/jena/assets/304786/56364cbf-7286-4769-99ed-668da87667a0)
   
   If you remove one of the files, it's reactive, so it adjusts the progress 
bar accordingly:
   
   
![image](https://github.com/apache/jena/assets/304786/26f4df5f-c48e-4caf-b381-9dc8fe0c6da5)
   
   And if you add a new file, it again updates correctly.
   
   
![image](https://github.com/apache/jena/assets/304786/5c3b80cc-a9d5-4710-b44e-44e01cd9ebdd)
   
   I think that will indeed help users who need to upload multiple files at 
once. Next step the tests (when I find time for that :+1:)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to