[CLOSED] Create a new command file.openDroppedFiles so that we can support folder drop on Brackets.

[core-ai-bot]

Issue by RaymondLim
Monday Oct 21, 2013 at 04:14 GMT
Originally opened as adobe/brackets#5590

---

This pull request is to consolidate different code that handle the drag and drop of files and folders by creating a new command for openDroppedFiles in DragAndDrop.js and calling it for all drag-and-drop triggered from the shell code. So it is part of the fix for issue #5015. There are two more pull requests from brackets-shell to complement this.

---

RaymondLim included the following code: https://github.com/adobe/brackets/pull/5590/commits

[core-ai-bot]

Comment by bchintx
Monday Oct 21, 2013 at 22:21 GMT

---

Done with initial review. Looks pretty straightforward.

@RaymondLim Was there another pull request (ie. in the shell) that you needed me to review along with this one?

[core-ai-bot]

Comment by RaymondLim
Monday Oct 21, 2013 at 22:32 GMT

---

@bchintx I'm still working on another pull request that also needs to fix #5586. But you can merge this one without causing any issue though. If you would like to review the upcoming pull request, then you can use rlim/handle-dropped-folder branch in brackets-shell repo. I had mac specific changes done in that branch. That branch also requires pr adobe/brackets-shell#353.

[core-ai-bot]

Comment by bchintx
Tuesday Oct 22, 2013 at 21:35 GMT

---

Looks good. Merging...