-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Reimplement camera gestures #230
Comments
Regarding keyboard:Usually one can...
Ctrl is usually only necessary for move and zoom when the map is embedded as an iframe within a website, i.e. more generally, if it is within a scrollable area. Regarding mouse:For right-mouse-button or Shift + mouse button, there is no expected standard behavior: Holding the right-mouse-button sometimes is used to rotate + tilt. But on maps that show a context menu on right-mouse-button click, the right mouse button usually has no gesture function. Instead, Shift + left-mouse-button is rotate + tilt. For both MapBox and MapLibre it is possible to select an area with Shift + left-mouse-button in which the view should zoom into. Ctrl is usually necessary for any mouse gesture when the map is embedded as an iframe within a website, i.e. more generally, if it is within a scrollable area. |
sgtm. for keyboard-only controls, we should probably consider focus.
we probably don't need to handle this, beyond making it easy to configure in our public API.
Investigating other maps:
I think I like the Mapbox defaults, but configuration API should allow for Google/Apple-like behavior |
Once we integrate with the native core, we'll lose SDK gestures on iOS, Android, and Desktop. Once we set
zIndex
to a negative value on web (to support Compose overlays), we'll lose SDK gestures there too.We should reimplement gestures in Compose with
pointerInput
. #216 has a draft implementation of this, with pan, zoom, and rotate. This issue serves as a place to note requirements for the real version we'll build after #218. It's the ideal goal; unsure how much will be possible in Compose.cc @westnordost did I miss anything?
The text was updated successfully, but these errors were encountered: