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

Location Button #862

Draft
wants to merge 23 commits into
base: v.next
Choose a base branch
from
Draft

Location Button #862

wants to merge 23 commits into from

Conversation

rolson
Copy link
Contributor

@rolson rolson commented Sep 6, 2024

Adds a button to the toolkit that allows a user to show their current location.

The view can be positioned anywhere, with any background. This example uses a "regular material background" in a rounded rectangle.

  • Tap once, location display starts, if using a system location datasource, it will request authorization, while starting a progress indicator is shown
  • Starts in auto pan mode recenter
  • Long press to get a context menu
  • Switch to navigation mode, any autopan mode, or turn the datasource off
  • If the location display fails to start, an warning signal is shown, tapping again will try to start the datasource again
  • Panning turns autopan off, tapping the button again will turn on the last autopan mode that the user had selected (barring the off value)

More details

  • The context menu is only shown on long press if the count of the auto pan options the user passes in is greater than 1
  • The "last selected auto pan mode" starts out with a default value of the first value passed to the initializer in the autoPanOptions that matches a value in this list: .recenter, .compassNavigation, .navigation
Location Display Starting Recenter auto pan mode Long Press Context Menu Navigation Mode Location Display failed to start

@rolson
Copy link
Contributor Author

rolson commented Sep 9, 2024

Next sprint x I can write some tests for this and then we can release it PI1 of 2025.

@rolson
Copy link
Contributor Author

rolson commented Sep 9, 2024

cc @dg0yal

@dfeinzimer dfeinzimer mentioned this pull request Sep 13, 2024
@rolson
Copy link
Contributor Author

rolson commented Dec 15, 2024

Testing, doc, and tutorial now completed.

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

Successfully merging this pull request may close these issues.

1 participant