-
Notifications
You must be signed in to change notification settings - Fork 18k
x/mobile/app: panic with palm touch on iOS app #13435
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
/cc @crawshaw Is the following comment no longer true? // It is widely reported that the iPhone can handle up to 5 simultaneous |
patch & log diff --git a/app/darwin_armx.go b/app/darwin_armx.go
index 5344e27..f6fa027 100644
--- a/app/darwin_armx.go
+++ b/app/darwin_armx.go
@@ -132,6 +132,7 @@ var touchEvents struct {
//export sendTouch
func sendTouch(cTouch, cTouchType uintptr, x, y float32) {
+ log.Println(cTouch, cTouchType, x, y)
id := -1
for i, val := range touchIDs {
if val == cTouch {
device: iPhone5s |
6 fingers touch and release on iPhone5S. expect:
instead:
No problem if less than 6 fingers. |
need handle touchesCancelled ? |
CL https://golang.org/cl/18180 mentions this issue. |
No occur panic in Android, but there was a problem of lost touch sequence (similar IOS).
Proposal:
|
improved a problem that continues to maintain a touch-ID that must be discarded by 'touchesCanceled' event. Fixes golang/go#13435 Change-Id: I13635255e5317bba2202bf7a8d2dd957f7824d0a Reviewed-on: https://go-review.googlesource.com/18180 Reviewed-by: David Crawshaw <crawshaw@golang.org>
improved a problem that continues to maintain a touch-ID that must be discarded by 'touchesCanceled' event. Fixes golang/go#13435 Change-Id: I13635255e5317bba2202bf7a8d2dd957f7824d0a Reviewed-on: https://go-review.googlesource.com/18180 Reviewed-by: David Crawshaw <crawshaw@golang.org>
golang 1.5.1
https://github.com/golang/mobile/blob/master/app/darwin_armx.go#L151
The text was updated successfully, but these errors were encountered: