refactor: let the target be passed into dragStart

[zsoerenm]

The event.target changes in asynchronous calls. It is therefore necessary to save the target at the desired time instance and pass it to dragStart.

Fixes: #1032

const target = e.target;
moveable.waitToChangeTarget().then(() => {
         moveable.dragStart(e, target);
   });

is an asynchronous call. e.target can, therefore, change. Fixing the target helps.

[zsoerenm]

@daybrush
I added a second commit (8bb8293) that lets dragStart default to currentTarget instead of target.
This is especially useful for elements that consists of multiple sub-elements.
EDIT: This second commit is a breaking change. So you might want to merge only the first commit.

[zsoerenm]

I dropped the latest commit, because it is a breaking change and it fails tests. The currentTarget can by passed to dragStart manually anyway and, I think, it is better to not introduce a breaking change here.

[daybrush]

@zsoerenm

It's good code. I will reflect it.

[zsoerenm]

@daybrush
would you mind making a new release?

[daybrush]

@zsoerenm

Maybe release by monday of this week.

[daybrush]

@zsoerenm

• croact-moveable 0.9.0
• @moveable/helper 0.1.3
• lit-moveable 0.30.0
• moveable 0.53.0
• preact-moveable 0.55.0
• react-moveable 0.56.0
• svelte-moveable 0.45.0
• vue-moveable 2.0.0-beta.87
• vue3-moveable 0.28.0
• ngx-moveable 0.50.0

moveable's new version is released. Check it again.

[zsoerenm]

Thank you!