Skip to content

MapControl: Added IP based location, fixed map centering and zoom, enhanced leaflet - #63

Merged
brYch97 merged 2 commits into
TALXIS:masterfrom
zdeneksrejber:users/zdenek.srejber/mappicker-fixes
Aug 25, 2026
Merged

MapControl: Added IP based location, fixed map centering and zoom, enhanced leaflet#63
brYch97 merged 2 commits into
TALXIS:masterfrom
zdeneksrejber:users/zdenek.srejber/mappicker-fixes

Conversation

@zdeneksrejber

@zdeneksrejber zdeneksrejber commented Aug 19, 2026

Copy link
Copy Markdown
Member
  • Falls back to an IP-based location guess when no dataset locations are available
  • Auto-zooms to fit a single location
  • Auto-calculates bounds to fit all locations when there are multiple
  • Applies equivalent improvements to the Leaflet provider and its sample dataset handling in Storybook

- Introduced a custom marker icon for map pins with specific dimensions and anchor points.
- Added fallback center and zoom level for the map when no locations are provided.
- Updated FitToLocations component to adjust padding for better visibility.
- Modified LeafletMap component to dynamically set the center and zoom based on provided locations or fallback values.
- Enhanced MapDemo component to include a toggle for displaying sample dataset pins.
- Updated useSampleMapDataset hook to conditionally return sample locations based on the new showPins parameter.
- Adjusted sample location coordinates for improved accuracy.
@zdeneksrejber
zdeneksrejber requested a review from a team August 19, 2026 14:27
@zdeneksrejber zdeneksrejber changed the title MapControl: Fixed map centering and zoom, enhanced leaflet MapControl: Added IP based location, fixed map centering and zoom, enhanced leaflet Aug 20, 2026
@zdeneksrejber
zdeneksrejber force-pushed the users/zdenek.srejber/mappicker-fixes branch from 6f0d617 to 7018b8b Compare August 20, 2026 14:16
@brYch97
brYch97 merged commit e989c75 into TALXIS:master Aug 25, 2026
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