Skip to content
This repository has been archived by the owner on May 24, 2021. It is now read-only.

Maps lib class #45

Merged
merged 4 commits into from
Feb 11, 2015
Merged

Maps lib class #45

merged 4 commits into from
Feb 11, 2015

Conversation

derekeder
Copy link
Owner

Modifications to @amenadiel's pull request #42

@derekeder
Copy link
Owner Author

Merging!

derekeder added a commit that referenced this pull request Feb 11, 2015
@derekeder derekeder merged commit d85cf78 into master Feb 11, 2015
@derekeder
Copy link
Owner Author

Thanks @amenadiel! This is a great addition! Very nice to be able to customize a new map without having to dive into maps_lib.js

Now it's as easy as:

          var myMap = new MapsLib({
            fusionTableId:      "1m4Ez9xyTGfY2CU6O-UgEcPzlS0rnzLU93e4Faa0",
            googleApiKey:       "AIzaSyA3FQFrNr5W2OEVmuENqhb2MBB2JabdaOY",
            locationColumn:     "geometry",
            map_center:         [41.8781136, -87.66677856445312],
            locationScope:      "chicago",
          });

@derekeder
Copy link
Owner Author

@amenadiel also, you'll notice I deleted index_amd.html. It didn't seem distinct enough to warrant including. For those who like require.js, I'm ok with them figuring out how to use it themselves.

@ffflabs
Copy link
Contributor

ffflabs commented Feb 11, 2015

No problem. The example meant to justify the inclusion of the optional define call.

@derekeder derekeder deleted the maps-lib-class branch February 23, 2015 20:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants