Skip to content

Feat/user location style #112

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
Merged
merged 3 commits into from
Oct 29, 2025
Merged

Feat/user location style #112

merged 3 commits into from
Oct 29, 2025

Conversation

@Archdoog
Copy link
Collaborator

@Archdoog Archdoog commented Oct 28, 2025 •
edited
Loading

Issue/Motivation

  • Adds user location annotation customization.
Screenshot 2025-10-27 at 5 20 22 PM Screenshot 2025-10-27 at 5 20 44 PM

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

@Archdoog Archdoog requested review from hactar, ianthetechie and scottrhoyt and removed request for scottrhoyt October 28, 2025 00:32
ianthetechie
ianthetechie approved these changes Oct 28, 2025
View reviewed changes
Copy link
Collaborator

@ianthetechie ianthetechie left a comment

Choose a reason for hiding this comment

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

Nice 🎉 LGTM. Will leave this open for a bit in case anyone else wants to review.

scottrhoyt
scottrhoyt approved these changes Oct 28, 2025
View reviewed changes
Copy link
Collaborator

@scottrhoyt scottrhoyt left a comment

Choose a reason for hiding this comment

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

Overall looks great! I'm looking forward to being able to customize this. I just had one comment on naming that might be good to address.

@Archdoog Archdoog merged commit ce0bf6e into maplibre:main Oct 29, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ianthetechie ianthetechie ianthetechie approved these changes

@scottrhoyt scottrhoyt scottrhoyt approved these changes

@hactar hactar Awaiting requested review from hactar

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Archdoog @ianthetechie @scottrhoyt