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

Aux window: drag and drop shows invalid drop feedback outside window and sometimes fails on macOS #198128

Closed
gjsjohnmurray opened this issue Nov 13, 2023 · 9 comments
Assignees
Labels
*as-designed Described behavior is as designed help wanted Issues identified as good community contribution opportunities under-discussion Issue is under discussion for relevance, priority, approach workbench-auxwindow Issues related to use of auxiliary ("floating") windows. workbench-dnd Drag-and-drop issues
Milestone

Comments

@gjsjohnmurray
Copy link
Contributor

Type: Bug

Testing #8171 on Windows 11, I was disconcerted to see the "no drop" pointer (red circle, red diagonal slash through it). But when I dropped, the aux window opened correctly.

VS Code version: Code - Insiders 1.85.0-insider (345c8d0, 2023-11-13T05:39:23.476Z)
OS version: Windows_NT x64 10.0.22621
Modes:

@gjsjohnmurray
Copy link
Contributor Author

/assign @bpasero

@bpasero
Copy link
Member

bpasero commented Nov 13, 2023

Unfortunately I do not see a way to address this without causing unwanted side effects on the target application that you are dropping, but if someone has suggestions, please speak up.

Essentially I want to indicate that dropping is possible but without the target application to accept the data (e.g. a terminal should not change the prompt with the data of the dragged tab).

Somewhat related: this also causes a noticeable delay on macOS until the window opens because on macOS there seems to be an animation first to move the dragged tab to where it was:

Recording 2023-11-13 at 17 18 46

@bpasero bpasero added help wanted Issues identified as good community contribution opportunities question-discussion Question that could be a discussion topic (experimental label) workbench-auxwindow Issues related to use of auxiliary ("floating") windows. under-discussion Issue is under discussion for relevance, priority, approach and removed question-discussion Question that could be a discussion topic (experimental label) labels Nov 13, 2023
@bpasero bpasero added this to the Backlog milestone Nov 13, 2023
@bpasero bpasero added the workbench-dnd Drag-and-drop issues label Nov 13, 2023
@bpasero bpasero modified the milestones: Backlog, On Deck Nov 13, 2023
@bpasero
Copy link
Member

bpasero commented Nov 20, 2023

Btw I cannot reproduce the "no drop" pointer when dropping over desktop:

Recording 2023-11-20 at 17 03 44

@gjsjohnmurray
Copy link
Contributor Author

Desktop still gives me the no-drop indicator using this morning's Insiders:

junk

@bpasero
Copy link
Member

bpasero commented Nov 20, 2023

Is that stable VS Code or insiders? I am not sure, but the logo looks blue.

@gjsjohnmurray
Copy link
Contributor Author

It was Insiders.

@bpasero bpasero changed the title Aux window: drag/drop tab onto desktop works but shows a "no drop" pointer Aux window: drag and drop shows invalid drop feedback outside window Nov 29, 2023
@bpasero bpasero changed the title Aux window: drag and drop shows invalid drop feedback outside window Aux window: drag and drop shows invalid drop feedback outside window and sometimes fails on macOS Nov 29, 2023
@bpasero
Copy link
Member

bpasero commented Nov 29, 2023

On macOS this is even worse because there is an ugly 500-1000ms animation for invalid drops that causes all sorts of issues:

  • the entire operation feels sluggy and slow
  • if you manage to move the mouse back over the main window shortly after releasing, we cancel the operation assuming you wanted to drop over the main window

@bpasero bpasero added the *as-designed Described behavior is as designed label Dec 1, 2023
@VSCodeTriageBot
Copy link
Collaborator

The described behavior is how it is expected to work. If you disagree, please explain what is expected and what is not in more detail. See also our issue reporting guidelines.

Happy Coding!

@VSCodeTriageBot VSCodeTriageBot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 1, 2023
@bpasero
Copy link
Member

bpasero commented Dec 1, 2023

For now this will not be changed until maybe we change to an entirely different solution that involves a more complicated approach without drag and drop. The macOS delay issue will be addressed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*as-designed Described behavior is as designed help wanted Issues identified as good community contribution opportunities under-discussion Issue is under discussion for relevance, priority, approach workbench-auxwindow Issues related to use of auxiliary ("floating") windows. workbench-dnd Drag-and-drop issues
Projects
None yet
Development

No branches or pull requests

3 participants