Ticket #3132 (closed feature: fixed)

Opened 2 years ago

Last modified 2 years ago

evt.xy has incorrect positions after scrolling on mobile devices

Reported by: ahocevar Owned by: tschaub
Priority: minor Milestone: 2.11 Release
Component: Events Version: SVN
Keywords: mobile Cc:
State: Commit

Description (last modified by ahocevar) (diff)

This behavior can be observed on the iPhone 4. Maybe an issue with Event::getMousePosition?

Attachments

openlayers-3132.patch Download (1.1 KB) - added by ahocevar 2 years ago.

Change History

  Changed 2 years ago by ahocevar

  • description modified (diff)

Changed 2 years ago by ahocevar

follow-up: ↓ 3   Changed 2 years ago by ahocevar

  • state set to Review

The attached patch fixes the issue. The manual page-position.html test works as expected now on the iPhone, with no side effects on other browsers. Thanks for any review.

in reply to: ↑ 2   Changed 2 years ago by erilem

Replying to ahocevar:

The attached patch fixes the issue. The manual page-position.html test works as expected now on the iPhone, with no side effects on other browsers. Thanks for any review.

Andreas, do we really want to keep the OpenLayers.Console.log?

  Changed 2 years ago by ahocevar

@elemoine: no, that was not meat to stay.

  Changed 2 years ago by erilem

  • state changed from Review to Commit

Andreas. Please commit. For my information, your comment mentions getBoundingClientRect, which I thought was an IE thing?

  Changed 2 years ago by ahocevar

Eric: To my knowledge, IE was the first browser that implemented getBoundingClientRect, but now most browsers have it.

  Changed 2 years ago by ahocevar

  • status changed from new to closed
  • resolution set to fixed

(In [11706]) fixing scroll position issue on the IPhone by only clearing the mouse cache conditionally. r=erilem (closes #3132)

Note: See TracTickets for help on using tickets.