Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ui for changing pitch #1063

Closed
ansis opened this issue Mar 6, 2015 · 8 comments
Closed

ui for changing pitch #1063

ansis opened this issue Mar 6, 2015 · 8 comments
Labels
feature 🍏

Comments

@ansis
Copy link
Contributor

ansis commented Mar 6, 2015

How can a user change the pitch of a map? A button? or mouse action?
@tmcw
Copy link
Contributor

tmcw commented Mar 6, 2015

One vote for a button, at least at first, for normal pitch and some nice 45° or something.

@peterqliu
Copy link
Contributor

My first instinct is shift + left click drag to tilt up and down, and a button to restore to a full 90 degrees.

How would we handle touchscreens?

@ansis ansis mentioned this issue Apr 15, 2015
Closed
@peterqliu
Copy link
Contributor

On second thought, we already rotate around the Z axis with right-click and drag.

Let's make the right mouse button the center of all rotations: drag to rotate around Z axis, shift-drag to rotate around the X axis (adjusting pitch).

@atlefren
Copy link

Hi guys, i wanted a way to illustrate the pitch change, so I went ahead and made a (rater rudimentary) button added to the navigation control, inspired by Google Earth, If it's of any interest check out https://github.com/Norkart/mapbox-gl-js/tree/tilt-control2 and let me know if you want a pull request on this

scothis added a commit that referenced this issue Oct 27, 2015
@scothis
Add UI controls for pitch
f7ac991 
Use shift-up-arrow and shift-down-arrow to control the pitch. Clicking on
the rotation button will reset the pitch to 0.

This change exposes pitch for power users, but leaves the current UI
unchanged. More advanced UIs can still be built on top of this behavior.

Issue: #1063
@scothis scothis mentioned this issue Oct 27, 2015
Merged
@bhousel
Copy link
Contributor

bhousel commented Jan 13, 2016

We should support two finger swipe up/down for adjusting pitch, this has become a standard.

touch_zoom_rotate_pitch?

We really should just name that handler touch.

@lucaswoj
Copy link
Contributor

We have a keyboard shortcut for changing pitch. If we decide to add additional interactions or a UI element, let's ticket out that separately.

@fedex1
Copy link

fedex1 commented Jul 31, 2018

@bhousel Did your suggestion of "two finger swipe up/down for adjusting pitch" ever make it live? I do not see an easy way to tilt the map on mapboxgl-js mobile. Is there a way to tilt using finger gestures?

@bhousel
Copy link
Contributor

bhousel commented Jul 31, 2018

Sorry @fedex1 - I am not working on mapbox-gl-js these days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature 🍏
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@tmcw @bhousel @fedex1 @lucaswoj @ansis @atlefren @peterqliu