Skip to content

Conversation

AntoLC
Copy link
Collaborator

@AntoLC AntoLC commented Sep 2, 2025

Purpose

We have different DND system in the page, one on the menu tree and one in the Blocknote editor.
The menu tree was adding a transparent layer when user were dragging element on the Blocknote editor, blocking the Blocknote DND.

Proposal

We update the ui-kit to add the prop dndRootElement (suitenumerique/ui-kit#106), dndRootElement is now used to specify the root element for DND, this transparent layer is now only applied when a drag is made from the menu tree.

It stabilize as well the drop position, making it easier to drop element.

  • 🐛(frontend) fix background color during dnd
  • 🐛(frontend) fix dnd conflict with tree and Blocknote

@AntoLC AntoLC self-assigned this Sep 2, 2025
@AntoLC AntoLC added bug Something isn't working frontend multipages labels Sep 2, 2025
@AntoLC AntoLC linked an issue Sep 2, 2025 that may be closed by this pull request
Copy link

github-actions bot commented Sep 2, 2025

Size Change: -43 B (0%)

Total Size: 3.64 MB

Filename Size Change
apps/impress/out/_next/static/css/f6e4955e4e314bf8.css 0 B -25.1 kB (removed) 🏆
apps/impress/out/_next/static/d9d74de9/_buildManifest.js 0 B -869 B (removed) 🏆
apps/impress/out/_next/static/75870dd7/_buildManifest.js 867 B +867 B (new file) 🆕
apps/impress/out/_next/static/css/097de668257d29b5.css 25.1 kB +25.1 kB (new file) 🆕

compressed-size-action

When we were dragging an item in the doc tree,
the background color was opaque, making it difficult
to see the underlying content. The cause was
that we were overriding the transparent background
color.
We have different DND system in the page, one on the
menu tree and one in the Blocknote editor.
The menu tree was adding a transparent layer
when user were dragging element on the Blocknote
editor, blocking the Blocknote DND.
We update the ui-kit to add the prop dndRootElement,
dndRootElement is now used to specify the root element
for DND, this transparent layer is now only applied
when a drag is made from the menu tree.

It stabilize as well the drop position, making it easier to drop element.
Recent upgrade of ui-kit removed the z-index for
the modal backdrop, causing it to be hidden behind
other elements. This commit restores the z-index
 to ensure the modal backdrop is displayed correctly.
@AntoLC AntoLC merged commit 814eb1f into main Sep 3, 2025
21 of 22 checks passed
@AntoLC AntoLC deleted the fix/dnd-conflict branch September 3, 2025 08:48
@AntoLC AntoLC mentioned this pull request Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend multipages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to move blocks
2 participants