Skip to content

Conversation

Sebobo
Copy link
Member

@Sebobo Sebobo commented Aug 6, 2025

What I did

Show drag & drop indicators in inline editor. Resolves: #3977

How I did it

  • Use a custom body collection class that works inside the iframe and reattaches on location changes or reloads
  • Added the missing CSS for the indicators from CKEditor to the Neos styles.

How to verify it

Neos.CKEditor.Drop.Indicator.Fix.mp4

@github-actions github-actions bot added the 9.0 label Aug 6, 2025
@Sebobo Sebobo requested a review from mhsdesign August 6, 2025 10:04
@Sebobo Sebobo self-assigned this Aug 6, 2025
@Sebobo Sebobo changed the base branch from 9.0 to 9.1 August 6, 2025 11:20
@github-actions github-actions bot added 9.1 and removed 9.0 labels Aug 6, 2025
Sebobo added 2 commits August 6, 2025 13:21
The inline editor uses various elements for highlights which don’t work in the host frame.
With this patch, we instead attach the collection to the guestframe body.
With this change, the user can see an indicator where selected elements in the inline editor would show up.
The drag & drop feature worked before, but this makes the feature much more usable and intuitive.
It even works between multiple editor instances.

Resolves: #3977
@Sebobo Sebobo force-pushed the bugfix/show-ck-editor-drop-targets branch from b71d3cf to 1f375ed Compare August 6, 2025 11:21
@Sebobo Sebobo moved this to In progress in Neos 9.1 UX Project Aug 7, 2025
@Sebobo Sebobo changed the title Bugfix/show ck editor drop targets BUGFIX: Show ck editor drop targets Aug 12, 2025
@Sebobo Sebobo changed the title BUGFIX: Show ck editor drop targets BUGFIX: Show drop targets when dragging text in the inline editor Aug 12, 2025
@github-actions github-actions bot added the Bug Label to mark the change as bugfix label Aug 12, 2025
@Sebobo Sebobo marked this pull request as ready for review August 12, 2025 12:54
@Sebobo Sebobo changed the title BUGFIX: Show drop targets when dragging text in the inline editor FEATURE: Show drop targets when dragging text in the inline editor Aug 19, 2025
@github-actions github-actions bot added the Feature Label to mark the change as feature label Aug 19, 2025
@Sebobo Sebobo force-pushed the bugfix/show-ck-editor-drop-targets branch from 65204a4 to 61d048a Compare August 19, 2025 07:35
@Sebobo Sebobo requested a review from markusguenther August 19, 2025 07:35
@Sebobo Sebobo moved this from In progress to In review in Neos 9.1 UX Project Aug 19, 2025
@Sebobo
Copy link
Member Author

Sebobo commented Aug 21, 2025

TODO: I think we should move the CK BodyCollection into our Neos backend container, so we have everything inside one element and could use layer styles and maybe a web component for these elements in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
9.1 Bug Label to mark the change as bugfix Feature Label to mark the change as feature
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

BUG: Drag & drop indicators are not shown in inline editor
2 participants