Skip to content

Conversation

@stewx
Copy link
Contributor

@stewx stewx commented Sep 11, 2025

Description

This adds a thousands separator (or whatever is locale-appropriate) to the circles on the map view.

How Has This Been Tested?

Viewing the map

Screenshots (if appropriate)

Before:

image

After:

image

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR.
  • I have confirmed that any new dependencies are strictly necessary.
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code
  • All code in src/services/ uses repositories implementations for database calls, filesystem operations, etc.
  • All code in src/repositories/ is pretty basic/simple and does not have any immich specific logic (that belongs in src/services/)

Please describe to which degree, if any, an LLM was used in creating this pull request.

...

@stewx stewx changed the title Format numbers on map view Format point count numbers on map view Sep 11, 2025
@alextran1502 alextran1502 changed the title Format point count numbers on map view fix: format point count numbers on map view Sep 12, 2025
@alextran1502 alextran1502 enabled auto-merge (squash) September 12, 2025 03:03
@alextran1502 alextran1502 merged commit 04c9531 into immich-app:main Sep 12, 2025
54 of 65 checks passed
jochbru pushed a commit to pixelunioneu/immich that referenced this pull request Sep 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants