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

[WIP] Fix #4088: Add 'Upload Files...' menu entries #4166

Closed
wants to merge 0 commits into from

Conversation

jankeromnes
Copy link
Member

@jankeromnes jankeromnes commented Jan 25, 2019

Fix #4088: Add an Upload Files... menu entries.

ToDo:

  • Add a hidden <input type="file" multiple> to Theia's DOM somewhere, and trigger it from the new menu entries
  • Find a way to take input.files (which is a FileList) and send it to FileTreeModel.upload() (which takes a DataTransferItemList:)

https://github.com/theia-ide/theia/blob/5156023a4b0c590b40485f82d5c798c6ef6f8dec/packages/filesystem/src/browser/file-tree/file-tree-model.ts#L191-L196

@jankeromnes
Copy link
Member Author

Then again, we might want to support uploading files on Firefox and other browsers too.

My go-to library for this has always been https://github.com/felixge/node-formidable (super light-weight, works extremely well). Maybe I'll look into implementing that into Theia.

@jankeromnes
Copy link
Member Author

However, as mentioned by @akosyakov in #4088 (comment), importing formidable would require filling out a Contribution Questionnaire and/or filing an "IPBug" or an "IPZilla Bug" with the Eclipse Foundation. More hoops to jump through! ⭕️⭕️⭕️

@akosyakov
Copy link
Member

@jankeromnes if library improves ux a lot when use it, we will help with jumping through all necessary hoops. For now we don’t wait for dependency CQs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants