Skip to content

[DURACOM-320] fix Drag-and-Drop Malfunction in submission form - #6095

Closed
FrankGameDev wants to merge 2 commits into
DSpace:mainfrom
4Science:task/main/DURACOM-487
Closed

[DURACOM-320] fix Drag-and-Drop Malfunction in submission form#6095
FrankGameDev wants to merge 2 commits into
DSpace:mainfrom
4Science:task/main/DURACOM-487

Conversation

@FrankGameDev

Copy link
Copy Markdown

References

Description

Updated the CDK drag-and-drop behavior from horizontal to mixedto enable dropping chips on wrapped rows

Instructions for Reviewers

List of changes in this PR:

  • Changed the cdkDropListOrientation from horizontal to mixed

How to test the fix

Open a submission form, try adding some authors to let the chips wrap (or resize the browser/use DevTools to achieve it), and then try dragging and dropping chips onto different rows

@lgeggleston lgeggleston added bug component: submission 1 APPROVAL pull request only requires a single approval to merge port to dspace-9_x This PR needs to be ported to `dspace-9_x` branch for next bug-fix release port to dspace-10_x This PR needs to be ported to `dspace-10_x` branch for next bug-fix release labels Aug 13, 2026
@lgeggleston lgeggleston moved this to 🙋 Needs Reviewers Assigned in DSpace 11.0 Release Aug 13, 2026
@lgeggleston

Copy link
Copy Markdown
Contributor

Thanks for the fix @FrankGameDev! Since this was fixed on 8.1, we can try to port this back to 9.x and 10.x as well.

@FrankGameDev FrankGameDev changed the title [DURACOM-487] fix Drag-and-Drop Malfunction in submission form [DURACOM-320] fix Drag-and-Drop Malfunction in submission form Aug 14, 2026
@FrankGameDev

Copy link
Copy Markdown
Author

Thanks for the fix @FrankGameDev! Since this was fixed on 8.1, we can try to port this back to 9.x and 10.x as well.

Here they are: #6096 #6097!

@FrankGameDev

Copy link
Copy Markdown
Author

I'm closing this PR, as i created the branch with the wrong name. Here it is the new one #6098

@github-project-automation github-project-automation Bot moved this from 🙋 Needs Reviewers Assigned to ✅ Done in DSpace 11.0 Release Aug 17, 2026
@FrankGameDev
FrankGameDev deleted the task/main/DURACOM-487 branch August 17, 2026 08:18
@lgeggleston

Copy link
Copy Markdown
Contributor

Hi! Thanks @FrankGameDev, sounds good.
Also just a note for future PRs, usually it's not necessary to create backports beforehand - I apologize for the confusing wording of this comment! The "port to" labels added here mean that a Github action to auto-port will be attempted when a PR is merged, which is usually the easiest method to try before manually backporting (and will save you from having to transfer any review changes to the other PRs).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge bug component: submission port to dspace-9_x This PR needs to be ported to `dspace-9_x` branch for next bug-fix release port to dspace-10_x This PR needs to be ported to `dspace-10_x` branch for next bug-fix release

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Drag-and-Drop Malfunction in submission form

2 participants