Skip to content

Conversation

scottrhoyt
Copy link
Collaborator

Issue/Motivation

It's useful to know when the user tracking mode has changed to do things like update the state of the UI. This PR forwards the underlying MLNMapViewDelegate's method to a closure provided by a modifier on MapView.

Tasklist

  • Include tests (if applicable) and examples (new or edits)
  • If there are any visual changes as a result, include before/after screenshots and/or videos
  • Add #fixes with the issue number that this PR addresses
  • Update any documentation for affected APIs

@scottrhoyt scottrhoyt force-pushed the tracking_mode_changed_modifier branch from 71f83a4 to 6724a86 Compare August 13, 2025 23:12
@scottrhoyt scottrhoyt force-pushed the tracking_mode_changed_modifier branch from 6724a86 to c6031f1 Compare August 13, 2025 23:16
Copy link
Collaborator

@Archdoog Archdoog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. Gonna add a couple more reviewers to it.

@ianthetechie
Copy link
Collaborator

LGTM as well. Just need to fix the formatting. I'm away from my computer till Monday but can fix it then if necessary.

@Archdoog Archdoog merged commit 327a93f into maplibre:main Sep 5, 2025
2 checks passed
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.

3 participants