[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: |
|
||||||||||||||||||||||||||||||
| Sub-Tasks: |
|
||||||||||||||||||||||||||||||
| 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: | |||||||||||||||||||||||||||||||
| Work Started: | |||||||||||||||||||||||||||||||
| Description |
Expected resultsI am able to enable drag & drop, but disable moving before/after via a config if backend doesn't support it. Actual resultsI can only disable or enable drag&drop to all positions. WorkaroundDisable drag&drop and use move action with dialog instead. Development notesThe 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 |