[MGNLUI-7731] As developer I am able to enable drop ON_TOP but disable BEFORE/AFTER Created: 12/Jan/23  Updated: 13/Feb/23  Resolved: 07/Feb/23

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: None
Fix Version/s: 6.3.0

Type: Improvement Priority: Neutral
Reporter: Roman Kovařík Assignee: Quach Hao Thien
Resolution: Done Votes: 0
Labels: VN-Implementation
Σ Remaining Estimate: Not Specified Remaining Estimate: Not Specified
Σ Time Spent: 6.5h Time Spent: 6.5h
Σ Original Estimate: Not Specified Original Estimate: Not Specified

Issue Links:
dependency
Sub-Tasks:
Key
Summary
Type
Status
Assignee
MGNLUI-7785 Implementation Sub-task Completed Quach Hao Thien  
MGNLUI-7786 Review Sub-task Closed Roman Kovařík  
MGNLUI-7787 PreintQA Sub-task Completed Jaromir Sarf  
MGNLUI-7788 QA Sub-task Closed Antonín Juran  
MGNLUI-7822 docu review Sub-task Completed Adrian Brooks  
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: 3
Team: Nucleus
Work Started:

 Description   

Expected results

I am able to enable drag & drop, but disable moving before/after via a config if backend doesn't support it.

Actual results

I can only disable or enable drag&drop to all positions.

Workaround

Disable drag&drop and use move action with dialog instead.

Development notes

The constraint is hardcoded here https://git.magnolia-cms.com/projects/PLATFORM/repos/ui/browse/magnolia-ui-framework/src/main/java/info/magnolia/ui/contentapp/browser/drop/RowDragger.java#61, we could pass the definition with a configuration.



 Comments   
Comment by Roman Kovařík [ 06/Feb/23 ]

This was discussed here https://git.magnolia-cms.com/projects/PLATFORM/repos/ui/pull-requests/2297/overview.

I've removed the fix version 6.2.29. 

Comment by Quach Hao Thien [ 07/Feb/23 ]

Thank for reminding me ajuran,  I added you and abrooks to the docu PR, feel free to leave your comment to make it better

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