[MGNLUI-4360] Feedback of not allowed drag'n'drop node move is missing Created: 25/Jan/18  Updated: 08/Feb/23  Resolved: 08/Feb/23

Status: Closed
Project: Magnolia UI
Component/s: app framework
Affects Version/s: 5.5.8, 5.6.1, 5.7.1
Fix Version/s: 6.3.0, 6.2.29

Type: Improvement Priority: Neutral
Reporter: Frank Sommer Assignee: Antonín Juran
Resolution: Done Votes: 0
Labels: None
Σ Remaining Estimate: Not Specified Remaining Estimate: Not Specified
Σ Time Spent: Not Specified Time Spent: Not Specified
Σ Original Estimate: Not Specified Original Estimate: Not Specified

Attachments: PNG File image-2018-01-25-08-21-33-599.png    
Issue Links:
relation
Sub-Tasks:
Key
Summary
Type
Status
Assignee
MGNLUI-7773 Implementation Sub-task Completed Antonín Juran  
MGNLUI-7774 Review Sub-task Completed Adrian Brooks  
MGNLUI-7775 Pre-integration QA Sub-task Closed Roman Kovařík  
MGNLUI-7776 Final QA+PM Sub-task Completed Sang Ngo Huu  
MGNLUI-7836 Created MGNLUI-7831 Sub-task Closed Sang Ngo Huu  
Template:
Acceptance criteria:
Empty
Task DoD:
[X]* Doc/release notes changes? Comment present?
[X]* Downstream builds green?
[X]* Solution information and context easily available?
[X]* Tests
[X]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Date of First Response:
Epic Link: Issues with moving content item
Sprint: Nucleus 29
Story Points: 2
Team: Nucleus
Work Started:

 Description   

If you try to move a node with name 'a' in a folder where a node with the name 'a' already exists, the node moving won't be executed. It makes sense if the user get some feedback, that the move action is not allowed. Perhaps the destination is marked red before droped or a notification is displayed after dropping.



 Comments   
Comment by Antonín Juran [ 30/Jan/23 ]

Discovery

We could implement else block in TargetDataProviderUpdater#onDrop method showing an appropriate notification like:
...
else {
   Notification.show("The item couldn't be moved", Notification.Type.WARNING_MESSAGE);
}.
The TargetDataProviderUpdater is set in RowDragger ctor.

Comment by Sang Ngo Huu [ 08/Feb/23 ]

The QA is done with PM, but we agreed that the message is not detail/meaning enough, so MGNLUI-7831 is created to handle that

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