Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Remove draggable attribute in nodes.

See original GitHub issue

Do you want to request a feature or report a bug?

Now every node will wrap with a div container with draggable attribute, this will conflict with customized dnd features, source: https://github.com/ianstormtaylor/slate/blob/3339d088e14ad0bf04a674c14834179403ae369f/packages/slate-react/src/components/void.js#L79

Code source: https://github.com/Canner/slate-editor-icons/blob/master/packages/slate-editor-renderer/src/ImageNode.js#L128

You could try it live here: https://canner.github.io/slate-editor-icons/

What’s the current behavior?

Wrap every node with draggable attribute.

What’s the expected behavior?

The attribute should be remove.

Issue Analytics

State:
Created 5 years ago
Reactions:1
Comments:7 (4 by maintainers)

Top GitHub Comments

1reaction

ianstormtaylorcommented, Mar 26, 2018

I’m down for it to be opt-in. If whoever submits the PR can also make it so that the images and emoji examples remain draggable that would be perfect.

0reactions

ianstormtaylorcommented, Nov 28, 2019

I believe that this may be fixed by https://github.com/ianstormtaylor/slate/pull/3093, which has changed a lot of the logic in Slate and slate-react especially. I’m going to close this out, but as always, feel free to open a new issue if it persists for you. Thanks for understanding.