[TIMOB-25839] TiAPI: Touch event coordinate units do not match between platforms
| GitHub Issue | n/a |
|---|---|
| Type | Bug |
| Priority | Medium |
| Status | Closed |
| Resolution | Fixed |
| Resolution Date | 2019-01-11T02:28:58.000+0000 |
| Affected Version/s | Release 6.0.0 |
| Fix Version/s | Release 8.0.0 |
| Components | Android, iOS |
| Labels | android, breaking-change, defaultunit, ios, parity, touch |
| Reporter | Joshua Quick |
| Assignee | Joshua Quick |
| Created | 2018-03-06T03:44:25.000+0000 |
| Updated | 2019-01-11T02:29:03.000+0000 |
Description
*Summary:*
The (x,y) units provided by View event's "touchstart", "touchmove", "touchend", and "touchcancel" do not match between Android and iOS. Both platforms ignore the "ti.ui.defaultunit" property setting in "tiapp.xml".
On Android, the touch coordinates are always in "px" (ie: pixels).
On iOS, the touch coordinates are always in "dip" (aka: "dp").
*Steps to Reproduce:*
Build and run the below code on Android.
Drag your finger in the gray view.
Notice that the blue square does *not* correctly following your finger.
Build and run on iOS.
Drag your finger in the gray view.
Notice that the blue square *correctly* follows your finger.
Change "tiapp.xml" property "ti.ui.defaultunit" to "px".
Build and run on iOS.
Drag your finger in the gray view.
Notice that the blue square does *not* correctly following your finger.
Build and run on Android.
Drag your finger in the gray view.
Notice that the blue square *correctly* follows your finger.
function onTouch(event) {
if (event.source == containerView) {
touchView.center = { x: event.x, y: event.y };
}
}
var window = Ti.UI.createWindow(
{
layout: "vertical",
fullscreen: true,
theme: "Theme.AppCompat.NoTitleBar",
});
window.add(Ti.UI.createLabel(
{
text: "Touch Drag Test",
top: "20dp",
}));
var containerView = Ti.UI.createView(
{
touchEnabled: true,
backgroundColor: "gray",
top: "10dp",
bottom: "20dp",
width: "90%",
});
var touchView = Ti.UI.createView(
{
touchEnabled: false,
backgroundColor: "#008800",
width: "100dp",
height: "100dp",
});
containerView.add(touchView);
containerView.addEventListener("touchstart", onTouch);
containerView.addEventListener("touchmove", onTouch);
containerView.addEventListener("touchend", onTouch);
containerView.addEventListener("touchcancel", onTouch);
containerView.addEventListener("click", onTouch);
containerView.addEventListener("dblclick", onTouch);
containerView.addEventListener("doubletap", onTouch);
window.add(containerView);
window.open();
// Fetch the default unit property set in "tiapp.xml".
var defaultUnit = Ti.App.Properties.getString("ti.ui.defaultunit", "dip");
if (defaultUnit === "dp") {
defaultUnit = "dip";
} else if (defaultUnit === "system") {
defaultUnit = (Ti.Platform.name === "android") ? "px" : "dip";
}
// Do the below when a touch event has been received.
function onTouch(event) {
if (Ti.Platform.name === "android") {
event.x = Ti.UI.convertUnits(event.x + "px", defaultUnit);
event.y = Ti.UI.convertUnits(event.y + "px", defaultUnit);
} else if ((Ti.Platform.name === "iOS") || (Ti.Platform.name === "iPhone OS")) {
event.x = Ti.UI.convertUnits(event.x + "dip", defaultUnit);
event.y = Ti.UI.convertUnits(event.y + "dip", defaultUnit);
}
touchView.center = { x: event.x, y: event.y };
}
Hi have a similar issue using
On Android the value100is evaluated as100px, on iOS it is correctly evaluated as100dp.PR (master): https://github.com/appcelerator/titanium_mobile/pull/10414
[~Andrea.Vitale], I've written up a separate ticket for the
2DMatrix.translate()coordinate issue. Please see: [TIMOB-26563] Thanks for bringing this issue to our attention.*Closing ticket.* Fix verified in SDK Version
8.0.0.v20181129161342.Test and other information can be found at: https://github.com/appcelerator/titanium_mobile/pull/10414