Skip to content

Refactor Drag and Drop and implement sidenav and folder view dragging#1089

Merged
Rokt33r merged 1 commit intoBoostIO:masterfrom
Komediruzecki:feature/dnd-inside-folder-view
Aug 19, 2021
Merged

Refactor Drag and Drop and implement sidenav and folder view dragging#1089
Rokt33r merged 1 commit intoBoostIO:masterfrom
Komediruzecki:feature/dnd-inside-folder-view

Conversation

@Komediruzecki
Copy link
Copy Markdown
Contributor

@Komediruzecki Komediruzecki commented Aug 12, 2021

Add dnd inside folder view and across sidenav and folder view

Add callbacks for dnd functions
Update lower level components to use dnd functions

Add initial dnd refactor

Replace dragged resource with resource saving in transfer data
Change APIs across codebase (dragStart, dragEnd, dropOn)
Add text/plain data transfer to folder and doc dragg save events

Drag and drop showcase:

2021-08-12.15-52-30.mp4

Tested in desktop app

  • Test dropping a folder inside another folder (all views, only folders view)
  • Test dropping a note inside another folder
  • Test dropping a note/folder from folder view to sidebar (works)

Current bug (I think already present before)

  • Dropping in workspace/root folder gives already present but it is not
  • Dropping in lower level folders works

@Komediruzecki Komediruzecki requested a review from Rokt33r August 12, 2021 13:46
Copy link
Copy Markdown
Member

@Rokt33r Rokt33r left a comment

Choose a reason for hiding this comment

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

Dnd doesn't work between folder page and tree view in side navigator.

@Komediruzecki Komediruzecki force-pushed the feature/dnd-inside-folder-view branch from 5016bd6 to 0ff3659 Compare August 14, 2021 13:59
@Komediruzecki Komediruzecki requested a review from Rokt33r August 14, 2021 14:02
Add callbacks for dnd functions
Update lower level components to use dnd functions

Add initial dnd refactor

Replace dragged resource with resource saving in transfer data
Change APIs across codebase (dragStart, dragEnd, dropOn)
Add text/plain data transfer to folder and doc dragg save events
@Komediruzecki Komediruzecki force-pushed the feature/dnd-inside-folder-view branch from 0ff3659 to 155a8a0 Compare August 14, 2021 14:03
@Komediruzecki Komediruzecki changed the title Add dnd inside folder view Refactor Drag and Drop and implement sidenav and folder view dragging Aug 14, 2021
@Rokt33r Rokt33r merged commit c2763fa into BoostIO:master Aug 19, 2021
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