Buggy behaviour when pressing escape

HeyndrickxKarel commented 4 years ago

Hi

So I noticed an issue in the example Multiple lists with reorderable-group-for on your documentation.

Let's say, you are dragging "2" into the letters list and then cancel by pressing escape. Your lists will get stuck like the sreenshot below. Both lists will not reset to the previous state unless you start a new drag operation. It would make sense that the item that is being dragged is put back into its original list when pressing escape.
An other issue because of this occurs when you then drag the gray block inside one of the lists. This causes some replacements of items, some copies or just some buggy not-wanted behaviour.

To try these out in dumber gist you set up, you need to first click in preview window in order for the escape event to register.

HeyndrickxKarel commented 4 years ago

Nevermind, problem was just solved by clearing my cache. I suppose you have already fixed this :)

3cp commented 4 years ago

Dumber gist is bit troublesome for capturing esc. The embedded app window runs in an iframe, that's why you have to click it first (browser focus I guess).

The dnd doc site needs to be updated for the esc cancel feature. I have not done that. https://github.com/buttonwoodcx/doc-bcx-aurelia-dnd/issues/2

3cp commented 4 years ago

There are few more examples in the doc needs to be updated to handle cancellation too.

buttonwoodcx / bcx-aurelia-reorderable-repeat

Buggy behaviour when pressing escape #19