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

Moving the map does not disengage GPS (if no signal is available at the moment) #3432

Closed
mnalis opened this issue Oct 23, 2021 · 1 comment
Closed

Comments

@mnalis
Copy link
Member

mnalis commented Oct 23, 2021

If I understand UI correctly:

  • GPS icon is orange when GPS follow mode in enabled, ie. when map will be repositioned so center is on current GPS coordinates and follow that coordinates automatically as they change. It is black when GPS follow is disabled
  • one can enable GPS follow mode (and thus turn icon orange) by clicking on GPS icon when it is black
  • one can disable GPS follow mode (and thus turn icon black) by panning the map manually

It seems to work like that when one is in range of GPS satellites. However, when GPS signal is not acquired, it is not possible to disable GPS follow mode - ie. the GPS icon remain orange, and one cannot turn it black by panning the map manually.

I would expect I can disable GPS follow by same action (panning the map manually) even is GPS signal is not yet acquired.

How to Reproduce

  • Start SC inside building when there is no GPS signal
  • pan and rotate map around manually
  • notice the GPS icon remains orange all the time

(excuse the video quality - the GPS icon remains orange color all the time in the video below)

small_VID_20211023_025824.mp4

Versions affected
Android 6 / SC v37.0-beta1 github upstream

@mnalis mnalis added the bug label Oct 23, 2021
@westnordost westnordost self-assigned this Oct 23, 2021
@westnordost
Copy link
Member

Turns out this is deliberate. Git comment of the change that added this functionality:

panning the map doesn't disable location-following if no location is known yet
#2909 (comment)

@westnordost westnordost removed their assignment Oct 23, 2021
@westnordost westnordost removed the bug label Oct 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants