-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Multiselect support (#22) #41
base: master
Are you sure you want to change the base?
Conversation
I did the best of my ability with the translations, I think everything looks good but I'd definitely like a second opinion because I'm not 100% sure @andersevenrud EDIT: Whoops, still need to fix the eslint errors real quick, I only focused on the translation messages. |
Might not have time to look at this until this weekend. But at a quick glance things are looking good! |
Sorry, but still haven't found time to look at this. But as I sat here suddenly something dawned on me that we should take a not of. Most drag and drop handlers are implemented in a way that expects a single object, not an array of objects. So if the goal is to support multiple dnd objects at once this needs a look-in. |
When you say "drag and drop handlers" are you referring to the browser API or something else? Because a cursory internet search shows that there are a lot of good ways to handle DnD upload for more than one file simultaneously. There's even an |
This is a follow-up to #23, with additional functionality and support/integration with os-js/osjs-gui#40.
Currently in a draft state because of the TODO on line 147, which needs the functionality from the MeeseOS commit (the
formatStatusMessage
function) converted into the various OS.js translations.I also still need to go back and fix a couple copy/paste eslint errors that I forgot to fix in the first commit :p