Issue metadata
Sign in to add a comment
|
Touch/Pen control in ESRI Javascript API 3.xx not working on Beta 68
Reported by
alex.ple...@gmail.com,
Jul 12
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.99 Safari/537.36 Steps to reproduce the problem: 1. Call https://developers.arcgis.com/javascript/3/samples/routetask_basic_servicearea/ on a Windows Desktop device with touch/pen control 2. Try to zoom or pan the map with touch/pen control 3. Do the same on Chrome 67 stable where it works. What is the expected behavior? Zooming and panning of the map should work What went wrong? Zooming and panning of the map aren't working Did this work before? Yes 67.0.3396.99 Does this work in other browsers? Yes Chrome version: 68.0.3440.59 Channel: beta OS Version: 10.0 Flash Version:
,
Jul 12
It seems that the same page is also broken in FF. Can someone from QA do a bisect?
,
Jul 12
,
Jul 13
Able to reproduce the issue on the reported chrome 67.0.3396.99,latest canary 69.0.3486.0 using Windows10,Ubuntu17.10. Below is the Manual bisect information for same. (Good behavior seen on dot build) Note: Unable to provide Mac behavior as of now we don't have touch screen. Bisect Info: ================ Good build: 67.0.3396.99 Bad build: 68.0.3397.0 CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/67.0.3396.0..68.0.3440.0?pretty=fuller&n=10000 Suspect: https://chromium.googlesource.com/chromium/src/+/edafb744ebc6bb84aed04dce5eb3a011b5fe38e6 Reviewed-on: https://chromium-review.googlesource.com/1055068 yawano: Please confirm the issue and help in re-assigning if it is not related to your change.Adding RB-Stable label for M-68 feel free to change it if not required. Thanks!
,
Jul 13
,
Jul 13
The problem was in the ESRI Javascript API. They still use a "document.createTouch" object which is deprecated and got removed with chrome 68.
,
Jul 13
I think the path is used only when screen magnification is enabled. Have you tested this with screen magnification on or off? If it's tested with off, I don't think my CL has caused this regression.
,
Jul 13
Screen magnification I refer is screen magnification on Chrome OS. The code path would never run on Windows platform.
,
Jul 16
Is this still an issue then? Do we have a a workaround issue?
,
Jul 19
Friendly ping to get an update on this issue as per C#9 & it is marked as RBS. Thanks..!
,
Jul 20
yawano@, just wondering we would be able to get this fix for M68 stable?
,
Jul 24
Sorry for this late. As I have commented, I don't think my CL has caused this regression. I don't think I can make progress here. Assigning this issue back.
,
Jul 25
,
Jul 27
Tested this issue and unable to reproduce on latest stable 68.0.3440.75 and Dev #69.0.3497.12 and latest chrome 70.0.3504.0 using Windows 10 and Ubuntu 17.10. @Reporter: Can you please check on latest chrome and let us know if issue still persists or not. You can download latest chrome builds here:" https://www.chromium.org/getting-involved/dev-channel ". Removing RB-Stable label. Thanks!
,
Jul 27
Adding Needs-Feedback label to it.
,
Jul 27
,
Aug 2
Closing this as there wasn't any feedback from the reporter. Also maybe the developers have updated the code not to use document.createTouch anymore. CC Ella who was involved in deprecating those APIs. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dtapu...@chromium.org
, Jul 12