-
Notifications
You must be signed in to change notification settings - Fork 24
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.
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
Center the map on the user's location #99
Comments
PR did not address some bugs, so I reverted. There seems to be this weird bug on Firefox that if a user selects "Not Now", the error function is never called: https://bugzilla.mozilla.org/show_bug.cgi?id=675533. Likewise, Chrome only supports geolocation through https, which we don't have on our custom domain (which is also a GH Pages site) right now: https://developers.google.com/web/updates/2016/04/geolocation-on-secure-contexts-only. We'll need to set that up first-- @nicoledaniels do you have our Namecheap creds to look into that? |
^ That Bugzilla thread is a... well, several cursewords. I'm personally of the mind to say screw it for a bug that just got fixed for future release after 5.5 years of explicitly refusing to implement part of a specification because the devs think Firefox is above it. That's the sort of crap that made me stop using Mozilla software. |
Yeah I'm totally cool with a Mozilla conditional that drops the prompt and uses Cathy for now. |
Still working on getting the SSL certs for Google Chrome, but should be coming soon which will fix the problem |
Center the Leaflet map on the user's location if provided. The zoom should increase if the user's location is provided. If the location is not provided or geolocation services fails, set the center of the map to Cathy's coordinates.
The text was updated successfully, but these errors were encountered: