Implement drag-and-drop of tasks to other tasks or calendars#224
Implement drag-and-drop of tasks to other tasks or calendars#224nicolad wants to merge 1 commit intonextcloud:vuefrom nicolad:drag-and-drop
Conversation
|
Hello herr @raimund-schluessler, I've added the same dependency as I took a look at angular version of drag and drop; but if you can provide your thoughts on what is expected from implementation - it will be nice. |
|
Hey Herr @nicolad 😉
Default sort order would basically mean to sort by multiple task properties. First by completed and the ambiguous ones then by due date etc. I will have a look later.
This is a good idea. I didn't know they use this library.
Most important would be to be able to move subtasks from one task to the other or make it a root task (because drag and drop is currently the only way to change or move subtasks). Dropping on a task would make the dragged task a subtask of the target task. If necessary, the calendar would also be changed. The other options would be to drop it on one of the collections (important, due today, completed). Then the task would be set as important, due today, or completed. Or you could also drop it on a calendar in the sidebar and change the calendar like that. The logic to actually change a task's properties is not yet implemented. I think in order to keep this PR clean, it is best to focus on the dnd logic and just log to the console which task has to be altered in which way e.g. as here: https://github.com/nextcloud/tasks/blob/vue/src/components/TheDetails.vue#L542 |
|
Thank you for your input, based on your description, it seems that |
Signed-off-by: Vadim Nicolai <contact@vadimnicolai.com>
This demo does exactly what is needed for the app. It works even better than what currently is implemented in master. I haven't had a look at the code, though. But I guess it should work. 🎉 |
|
@nicolad Since you are a member of NC now, could you create the branch in the Tasks repo, please? This makes it easier to checkout the branch. |
9aaf91e to
14738ca
Compare
|
Closing as #194 was merged into master and I am not sure if "vue-draggable-nested-tree" is the best option. |
No description provided.