GreenInfo-Network / caliparks.org

Mapping social media in parks and other open spaces
http://www.caliparks.org/
22 stars 3 forks source link

Replace Google Maps API / Adjust credits + branding #645

Closed mojodna closed 7 years ago

mojodna commented 8 years ago

with...Leaflet? OL3?

Now that we're no longer using StreetView (for trails) or directions, there's no reason to use the Google Maps API. This is mainly an annoyance because of the branding confusion it introduces.

danrademacher commented 7 years ago

Replace with latest Leaflet.

gregallensworth commented 7 years ago

I have confiirmed that Leaflet is known not to work with server-side rendering. which this application is. This is a longer-standing issue, noted on other projects. I tried and found not workable, various approaches including loading react-leaflet async in componentDidMount(), and other folks' libraries such as leaflet-react-universal.

If the largest issue is branding confusion, adding more prominent credits could be done readily.

This would be a more significant undertaking, and could cause the site loading to become more slow than it already is.

Per chat with Dan, we'll stick to the immediate need which is the branding, by adding something similar to L.Control.Credits which I often use for GreenInfo work. credits_mock

gregallensworth commented 7 years ago

I have a first cut up at http://caliparks-development.herokuapp.com/#explore

This appears on all maps.

@danrademacher What do you think?

gregallensworth commented 7 years ago

How about this?

http://caliparks-development.herokuapp.com/#explore

Much smaller initial footprint, expanding into a smaller version of the previous version. Also, GIN logo is now white.

credits-expanded

credits-collapsed

danrademacher commented 7 years ago

Ship it!

gregallensworth commented 7 years ago

https://www.caliparks.org/#explore

Done.