sitepoint-editors / accessible-drag-and-drop Goto Github PK

Hi,

It looks like a lot of the code for sitepoint here on Gitbhub is MIT licensed. Is this particular code licensed under MIT as well?

Thanks,
-Rich

Hello,
first thanks for that demos that helped me for a project.

Based on demo4a code I found that it is possible to include HTML stuff within the li: it is well displayed.
BUT: it can only be selected/grabbed when cliking on pure text, not on (let say) bold or italic parts.
It is because the (let say) bold element is recieved in document.addEventListener and it don't have the draggable attribute. I changed this function in that way:
// for HTML content in the li the event comes for the that entity // if not pure text. try to reach li parent var tmp = e.target; while (tmp.nodeName != "LI") { tmp = tmp.parentElement; }
and then replace e.target by tmp in the following code (if(tmp.getAttribute('draggable')) and tmp.getAttribute('aria-grabbed') == 'false').

It works fine for me (li are selectable everywhere) and draggable.
You may consider inserting it, probably after improving it.
Note: this code doesn't cover the multi-selection case. If you select as 2nd (or more) element a line by clicking a HTMLed part it is ignored. The same changes should be applied somewhere else.

Regards,