-
Notifications
You must be signed in to change notification settings - Fork 197
MINOR: Fix input handling in dynamic markers example. #2153
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2153 +/- ##
==========================================
+ Coverage 67.67% 67.73% +0.05%
==========================================
Files 298 299 +1
Lines 26387 26470 +83
Branches 5973 5990 +17
==========================================
+ Hits 17857 17929 +72
- Misses 8530 8541 +11
Continue to review full report at Codecov.
|
709377a
to
750a791
Compare
handlePick(mapView, markersDataSource, event.pageX, event.pageY); | ||
} | ||
}); | ||
let longTapTimer: number | undefined; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we move out the long-press event related logic out of the example to make it reusable and not complicate the example logic?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done. I've adapted the LongPressHandler class to support touch events.
Add touch screen support. Signed-off-by: Andres Mandado <[email protected]>
234d895
to
38212d1
Compare
38212d1
to
e1a0f8d
Compare
Signed-off-by: Andres Mandado <[email protected]>
e1a0f8d
to
f322077
Compare
Signed-off-by: Andres Mandado <[email protected]>
Add touch screen support.