search

springload / wagtaildraftail

🐦📝🍸 Draft.js editor for Wagtail, built upon Draftail and draftjs_exporter
https://pypi.python.org/pypi/wagtaildraftail
MIT License
24 stars 3 forks source link

Link editing and removal are unreliable #47

Closed tomdyson closed 6 years ago

tomdyson commented 7 years ago

See screencast. I haven't been able to establish a pattern!

draftail-link-edit

thibaudcolas commented 7 years ago

I think I have experienced this too, didn't manage to pinpoint the cause either. Will investigate.

jjanssen commented 7 years ago

It seems like the initial cause itself is a part of the prefixing icon and not being able to select the first letter when visible (the w in Toms screen). At that point the cursor will jump to the beginning of the icon which does not hold the link but still shows the dialog.

When you disable the icon this is partially the same behaviour except the 1st letter is now clickable / targetable. But also when the cursor is in front of the first letter, it shows the dialog, but does not respond to either Edit or Remove.

Update: DraftJS Link demo shows the same behaviour with cursor in front of the text.

loicteixeira commented 7 years ago

Regardless of what default is chosen, the configuration should be documented (or linked to the draft one?)

thibaudcolas commented 6 years ago

Moving this to https://github.com/springload/draftail/issues/109, as the issue isn't specific to WagtailDraftail.