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

macOS: Cannot open directories by dropping on dock anymore #39496

Closed
0ff opened this issue Dec 3, 2017 · 3 comments
Closed

macOS: Cannot open directories by dropping on dock anymore #39496

0ff opened this issue Dec 3, 2017 · 3 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug important Issue identified as high-priority verified Verification succeeded workbench-os-integration Native OS integration issues
Milestone

Comments

@0ff
Copy link

0ff commented Dec 3, 2017

  • VSCode Version: Code - Insiders 1.19.0-insider (5b0fa2e, 2017-12-01T06:07:42.474Z)
  • OS Version: Darwin x64 17.2.0
  • Extensions:
Extension Author (truncated) Version
EditorConfig Edi 0.11.1
vscode-standardjs che 1.2.0
nasc-touchbar fel 1.8.2
terraform mau 0.0.19
vscode-language-babel mgm 0.0.10
cpptools ms- 0.14.3
node-debug2 ms- 1.19.0
platformio-ide pla 0.8.2
project-snippets reb 0.5.0
vscode-react-native vsm 0.5.7
vscode-todo-highlight way 0.5.11
debug web 0.21.2
markdown-all-in-one yzh 0.11.2

(2 theme extensions excluded)


Steps to Reproduce:

  1. Install latest insider build
  2. Drag a directory on the dock icon (or the icon in /Applications) - you can't.
  3. Repeat with latest release, observe it working

Reproduces without extensions: Yes

The issue seems to be this PR here: #37106
As it doesn't even seem to fix the actual problem (see #25843) I propose reverting it so drag & drop keeps working.

@vscodebot vscodebot bot added the insiders label Dec 3, 2017
@vscodebot vscodebot bot added workbench workbench-dnd Drag-and-drop issues labels Dec 3, 2017
@bpasero
Copy link
Member

bpasero commented Dec 3, 2017

@0ff I can reproduce. how did you find out it could be caused by #37106

@joaomoreno fyi

@bpasero bpasero added workbench-os-integration Native OS integration issues and removed workbench workbench-dnd Drag-and-drop issues labels Dec 3, 2017
@bpasero bpasero added this to the November 2017 milestone Dec 3, 2017
@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug important Issue identified as high-priority labels Dec 3, 2017
@bpasero bpasero changed the title Cannot open directories by Drag&Drop anymore macOS: Cannot open directories by dropping on dock anymore Dec 3, 2017
@0ff
Copy link
Author

0ff commented Dec 3, 2017

As this also happens when VSCode is not running at all, I just diff'ed the Insider Info.plist against the Release Info.plist, then removed the recently added LSItemContentTypes, verified it worked again and then traced the change back to this PR.

Please not that the dock is not the only place where this happens (though it's arguably the most used one). This also happens when trying to drop on the application icon in the finder itself.

@joaomoreno
Copy link
Member

Reverted: #39541

Thanks for the thorough investigation!

@sandy081 sandy081 added the verified Verification succeeded label Dec 5, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug important Issue identified as high-priority verified Verification succeeded workbench-os-integration Native OS integration issues
Projects
None yet
Development

No branches or pull requests

4 participants