Only trigger geolocation permission prompt on explicit "my location" click

[Copilot]

On index.html, loading the map page auto-triggered Chrome mobile's location permission dialog, even though the user hadn't interacted with the "my location" button.

Root cause

• centerMapOnDeviceLocation() ran on page load and called centerMapOnDeviceLocationNavigator(), which invoked navigator.geolocation.getCurrentPosition() unconditionally.
• The existing navigator.permissions.query() check only short-circuited for the denied state; the default prompt state still fell through to getCurrentPosition(), surfacing the native permission dialog on every load.

Fix

• centerMapOnDeviceLocation() now checks the Permissions API result before deciding how to center the map on load:
  • granted → use navigator.geolocation directly (no dialog, since permission is already resolved).
  • anything else (prompt, denied, or Permissions API unsupported) → skip getCurrentPosition() entirely and fall back to IP-based geolocation (ipwho.is), silently, with no dialog.
• The "my location" button handler (requestDeviceLocation) is unchanged — clicking it still explicitly calls getCurrentPosition() and can prompt for permission, as intended.

function centerMapOnDeviceLocation(map) {
	if (navigator.geolocation && navigator.permissions && navigator.permissions.query) {
		navigator.permissions.query({ name: 'geolocation' })
			.then(function (permissionStatus) {
				if (permissionStatus && permissionStatus.state === 'granted') {
					return centerMapOnDeviceLocationNavigator(map);
				}
				return centerMapOnDeviceLocationIpwhois(map);
			})
			.catch(function () { return centerMapOnDeviceLocationIpwhois(map); })
			.catch(function () { setDefaultMapCenter(map); });
		return;
	}
	centerMapOnDeviceLocationIpwhois(map).catch(function () { setDefaultMapCenter(map); });
}

• Fixes Fix /index.html still request 🛰️ location permission & show dialog 🗯️ in Chrome mobile #29

[Copilot]

Pull request overview

This PR prevents the browser’s geolocation permission prompt from appearing automatically on index.html page load by only using navigator.geolocation when permission is already granted, and otherwise falling back to IP-based centering.

Changes:

• Update centerMapOnDeviceLocation(map) to query the Permissions API and only call getCurrentPosition() when the geolocation permission state is granted.
• For prompt / denied / Permissions API unsupported cases, skip navigator.geolocation entirely on load and fall back to ipwho.is, with a default-center fallback on failure.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.