feat: add Drag step type
#491
Conversation
|
@OrKoN @Lightning00Blade This is still a WIP. PTAL and see if the idea here makes sense. In theory, a drag should do both drawing and dragging (i.e. drag n drop API calls), but I don't believe actual 'dragging' works right now, only drawing) |
| */ | ||
| duration?: number; | ||
| } | ||
|
|
||
| export interface DragStep extends ClickAttributes, StepWithTarget { |
There was a problem hiding this comment.
I don't think there is value in a drag step based on coordinates. We should do the following instead #273 (comment)
There was a problem hiding this comment.
Rationale: 1) no one is going to define coordinates by hand so recording part is a must 2) coordinates are fragile and any change in the viewport/website will break the recorded drag.
If it's for the drawing use case, let's use maybe a different name but the point 1 still applies. Without Recording, it is difficult to use. Let's, therefore, re-consider the whole story starting with the use case.
There was a problem hiding this comment.
I agree. While designing this it didn't make sense to use coordinates for drag 'n drop.
One thing I wanted as a requirement for this feature is to ensure drag n' drop and "draw"ing were the same API. It's not reasonable to separate them since from the user point of view, they are the same motion. Because of this, it's fairly important we find the correct granularity so we can do this.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
No description provided.