Skip to content
This repository has been archived by the owner on Oct 4, 2021. It is now read-only.

[Ide] Improve DnD handling #4777

Merged
merged 8 commits into from
May 24, 2018
Merged

[Ide] Improve DnD handling #4777

merged 8 commits into from
May 24, 2018

Conversation

sevoku
Copy link
Member

@sevoku sevoku commented May 11, 2018

This fixes several DnD handling issues:

  • Bring back Workbench (incl. welcome screen) drop support (fixes Drag and Drop into welcome page does not work anymore #4403, partially reverts 67b943d)
  • Support opening multiple solutions
  • Allow dropping solution folders (will open first solution file in the dropped folder)
  • Use only internal viewers to open dropped files
  • Don't load projects solutions dropped into the documents notebook tab-bar are (ones sln as text file for editing)

@sevoku sevoku added this to the 7.6 milestone May 11, 2018
@sevoku sevoku requested a review from Therzok May 11, 2018 10:58
@sevoku sevoku requested a review from slluis as a code owner May 11, 2018 10:58
try {
if (Services.ProjectService.IsWorkspaceItemFile (filePath))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like to keep the old behavior here. If a user drops a solution file to the IDE, it is very likely the intention to open the solution, not edit the solution file. On the other hand, if the project file is being dropped, it is very likely the intention is to edit the project file (notice that IsWorkspaceItemFile will return true for solutions, but false for projects).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not IDE, it's only dropping into the document notebook tabbar, IDE (toolbar and anywhere outside of the editor, + welcome screen) will just open the solution (not in the editor). If I drop something into the tabbar, which is only this area:
grafik
I expect it to be opened in the editor.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually many other areas don't treat solution files correctly at all. For example the solution pad should open a dropped solution, but it doesn't anything with it. I'll fix it...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kapture 2018-05-14 at 12 28 11

I want to keep this behavior.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, got it, I missed the case when no document is open. Fixed with: 5eb56ba

* Open multiple solutions in one workspace
* Support dropping of solution folders

(copy behaviour from fb38d90)
This copies the welcome page behaviour when the
Ctrl or Cmd (on Mac) key is pressed.
@slluis
Copy link
Member

slluis commented May 24, 2018

Approved by QA.

@slluis slluis removed the qa-pending label May 24, 2018
@sevoku sevoku merged commit e7129e5 into master May 24, 2018
@sevoku sevoku deleted the workbench-dnd-fixes branch May 24, 2018 10:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drag and Drop into welcome page does not work anymore
3 participants