Skip to content

[75384] Section cannot be dragged and dropped outside of current view#23460

Merged
bsatarnejad merged 1 commit into
release/17.5from
75384-section-cannot-be-dragged-and-dropped-outside-of-current-view
Jun 1, 2026
Merged

[75384] Section cannot be dragged and dropped outside of current view#23460
bsatarnejad merged 1 commit into
release/17.5from
75384-section-cannot-be-dragged-and-dropped-outside-of-current-view

Conversation

@bsatarnejad
Copy link
Copy Markdown
Contributor

Ticket

https://community.openproject.org/wp/75384

What are you trying to accomplish?

Sections can now be dragged and dropped outside the visible area. The page automatically scrolls when dragging near the edges by registering the content wrapper as a scroll container for the drag-and-drop controller.

@bsatarnejad bsatarnejad self-assigned this Jun 1, 2026
@bsatarnejad bsatarnejad marked this pull request as ready for review June 1, 2026 07:06
@myabc myabc requested a review from Copilot June 1, 2026 11:03
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Enables dragging/dropping type form configuration sections beyond the currently visible area by providing an explicit scroll container for the Stimulus drag-and-drop controller and making the active list area scrollable.

Changes:

  • Wrapes the main (active) list content in a dedicated scroll container element and exposes it as a scrollContainer Stimulus target.
  • Adds styling for the new active list container to allow scrolling within the main area.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
frontend/src/global_styles/content/_types_form_configuration.sass Adds styles for the new active-list scroll container.
app/components/work_package_types/form_configuration_component.html.erb Introduces a scrollable wrapper div around the main content and registers it as the drag-and-drop scroll container target.

Comment thread frontend/src/global_styles/content/_types_form_configuration.sass
@bsatarnejad bsatarnejad merged commit b284121 into release/17.5 Jun 1, 2026
18 of 22 checks passed
@bsatarnejad bsatarnejad deleted the 75384-section-cannot-be-dragged-and-dropped-outside-of-current-view branch June 1, 2026 14:04
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

3 participants