[MGNLUI-4082] drag & drop can't move on root Created: 17/Nov/16  Updated: 13/Jan/17  Resolved: 10/Jan/17

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: 5.4.9, 5.5
Fix Version/s: 5.4.11, 5.5.1

Type: Bug Priority: Major
Reporter: Tomáš Gregovský Assignee: Khuong Nguyen
Resolution: Fixed Votes: 1
Labels: None
Remaining Estimate: 0d
Time Spent: 18.25d
Original Estimate: 1.5h

Attachments: PNG File Move_mod1.png    
Issue Links:
duplicate
is duplicated by MGNLUI-4093 Drag and Drop don't allow drop nodes ... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:
Sprint: Saigon 75, Saigon 76 (xmas), Saigon 77, Saigon 78
Story Points: 8

 Description   

Locally same as on demoauthor. In Chrome and also in Safari.

Want to move page to change their orders on root level of pages. Simply use drag&drop to change order of travel and sportstation but without result. Playing around a bit, I can move into some node (page) and also change order of subpages, but then I am not able to move something from subpages to root. Same problem when using 'move action/dialog'. Same issues seems to be in JCR browser. So once I moved sportstation into travel, I can't get it back to root!



 Comments   
Comment by Tom Wespi [ 17/Nov/16 ]

I can confirm this problem for 5.4.9.

When using the «Move Dialog» it is possible to move nodes in the root, but not with drag and drop.

Comment by Evzen Fochr [ 30/Nov/16 ]

It is happening in all apps even with AlwaysTrueDropConstraint. Check TreeViewDropHandler for possible source of issue.

Comment by Khuong Nguyen [ 12/Jan/17 ]

Hi mdrapela,

I verified again after I downloaded magnolia-enterprise-5.2.1-SNAPSHOT on nexus (Thu Jan 12 2017 01:29:37 GMT +0700(ICT))
And It worked well same I and everyone help PreInt before. Please help to get the latest build on nexus and help me check it again.

Comment by Mikaël Geljić [ 12/Jan/17 ]

Hey Martin, thanks for the additional details

Re: surgeon fingers (brilliant expression btw), I can only second that and it's a well-known UI issue, for which we partially depend/wait upon Vaadin's Grid/TreeGrid to offer DnD. UX-wise, different move interactions (two-stepped), as well as Undo/Redo were also once considered, but there are no concrete plans on these matters at the moment.

This issue was indeed only about DnD; for the move dialog, we plan to display the root node in the move dialog; see MGNLUI-3900. This was attempted few months back, though it required a bit too many hacks in the current state of things.

Generated at Mon Feb 12 09:13:03 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.