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.

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

Move arrowhead cursor to bottom 3rd position when direction up is switched on #898

Closed
valleyofdawn opened this issue Dec 28, 2018 · 6 comments
Labels
App Native application related issues enhancement Medium Should be added/fixed in the future

Comments

@valleyofdawn
Copy link

Feature

When it tracking mode and with direction up users are likely to care more about what is ahead than what is behind them. I suggest that when both north up and the record button are pressed that the arrowhead cursor be moved to the bottom 3rd or 4th position. There should be space enough left for the height graph to show, though.

@valleyofdawn valleyofdawn added enhancement Medium Should be added/fixed in the future App Native application related issues labels Dec 28, 2018
@HarelM
Copy link
Member

HarelM commented Feb 5, 2020

These are the two mapbox threads about this requirement:
mapbox/mapbox-gl-js#1740
mapbox/mapbox-gl-js#4268
There are some solutions there, I don't know which of them will work, all hacks since this is not implemented in mapbox yet.
Since I don't think this is relevant for all users (mainly not for me) I can add a configuration options as part of #1145, but I believe this only progresses us towards a complex and unfriendly app...
Please consider closing this feature request.

@zstadler
Copy link
Member

zstadler commented Feb 5, 2020

Having used today the tilt option of the vector maps while following the GPS, I can see a significant value for this request in this user flow.

@HarelM
Copy link
Member

HarelM commented Feb 5, 2020

I would like to avoid another configuration option.
Do you agree to change the bahavior without the option to turn it off: i.e. When GPS is tracking and following user heading the cursor should be at the bottom part?Assuming I can make it happen code wise.

HarelM added a commit that referenced this issue Feb 6, 2020
@HarelM
Copy link
Member

HarelM commented Feb 6, 2020

@valleyofdawn Please try and install the latest version 8.6.19 (or later) and let me know if this does the trick.
Things to test:

  1. Regular GPS following
  2. GPS following with fixed north up
  3. GPS following while standing still
  4. Other scenarios you have in mind

I have added an offset of 100 pixels, if you feel it should be lower let me know.

@HarelM
Copy link
Member

HarelM commented Feb 7, 2020

I think this is good enough from the test I made. It has its gliches when returning to center but overall I think it's a nice addition.

@HarelM HarelM closed this as completed Feb 7, 2020
@HarelM HarelM added this to the Version 8.6.30 milestone Feb 8, 2020
@zstadler
Copy link
Member

The new cursor position is an improvement in course-up follow mode, but there is still a significant part of the screen which hosts the area behind. It is especially evident when using map tilt to get even longer-range view.

I would prefer to see the cursor as low as 100 pixel or so from the bottom of the screen.

I also noticed that moving the cursor to the bottom is useful when the map tilted, not rotating, and the view is towards the next section of the planned route.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App Native application related issues enhancement Medium Should be added/fixed in the future
Projects
None yet
Development

No branches or pull requests

3 participants