Skip to content
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

Size spec added and new template added using BlurredLocation object #7

Merged
merged 32 commits into from
Jun 16, 2017
Merged
Changes from 1 commit
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
f6921cb
new example using object added
mridulnagpal May 24, 2017
d7696ad
size spec added
mridulnagpal May 24, 2017
d7a4617
small tweaks
mridulnagpal May 24, 2017
b78d5ea
README updated
mridulnagpal May 24, 2017
c29e827
README updated
mridulnagpal May 24, 2017
138b289
Update README.md
mridulnagpal May 24, 2017
d05e6ef
Update README.md
mridulnagpal May 24, 2017
ab651be
Update README.md
mridulnagpal May 24, 2017
f70fc71
README updated
mridulnagpal May 25, 2017
42c8f43
README updated
mridulnagpal May 25, 2017
0124158
README updated
mridulnagpal May 25, 2017
b7b9ac7
Size removed
mridulnagpal May 25, 2017
1a486dc
options added
mridulnagpal May 26, 2017
b62fa04
object.js changed
mridulnagpal Jun 5, 2017
51d742c
merge conflict
mridulnagpal Jun 5, 2017
4c0c13a
addgrid working
mridulnagpal Jun 5, 2017
32cb2fb
README updated
mridulnagpal Jun 5, 2017
363190f
tweaks
mridulnagpal Jun 5, 2017
8bfa37c
changes in Gruntfile
mridulnagpal Jun 13, 2017
0af7deb
js file updated
mridulnagpal Jun 13, 2017
655845e
renaming examples
mridulnagpal Jun 13, 2017
f1d554e
Test passing
mridulnagpal Jun 15, 2017
81efadc
Structuring the library
mridulnagpal Jun 15, 2017
e345dca
Restructuring dist/built.js
mridulnagpal Jun 15, 2017
0c49afb
Renaming and using geocode function inside of panMapToGeocodedLocation()
mridulnagpal Jun 15, 2017
54cb90e
Removing redundancies
mridulnagpal Jun 15, 2017
1bbab6b
Renaming function
mridulnagpal Jun 15, 2017
c289926
Little tweaks to clean code
mridulnagpal Jun 15, 2017
f5fd39a
tweaks
mridulnagpal Jun 15, 2017
9b3507f
tweaks
mridulnagpal Jun 15, 2017
d51c63c
Making global methods internal
mridulnagpal Jun 16, 2017
642ab51
Test passing
mridulnagpal Jun 16, 2017
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 6 additions & 5 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,9 @@ To add new tests, edit the `*_spec.js` files in `/spec/javascripts/`.

## Options

| Methods | Use | Usage (Example) |
|---------|-----|-----------------|
|getLat | Used to get the current latitude of the center of the map. | blurredLocation.getLat() //This would return the value in numerics |
|getLon | Used to get the current latitude of the center of the map. | blurredLocation.getLon() //This would return the value in numerics |
|goTo | Takes in three parameters, namely latitude, longitude and zoom. Will set the center of map to co-ordinates input.| blurredLocation.goTo(44.51, -89.99, 13) //Will set center of map to (44.51,-89.99) with zoom set as 13 |
| Methods | Use | Usage (Example)|
|---|---|---|---|---|
|getLat |Used to get the current latitude of the center of the map.| blurredLocation.getLat() //This would return the value in numerics|
|getLon|Used to get the current latitude of the center of the map|blurredLocation.getLon() //This would return the value in numerics|
|goTo |Takes in three parameters, namely latitude, longitude and zoom. Will set the center of map to co-ordinates input.|blurredLocation.goTo(44.51, -89.99, 13) //Will set center of map to (44.51,-89.99) with zoom set as 13|
|getSize |Used to get the current size of the map, namely the width and height.| blurredLocation.getSize() //This would return the size of the map|
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, and I assume it's needed for some of the other functions. But let's keep in mind what is essential API that cannot be provided by direct access to the map, such as a native Leaflet call (not sure but probably map.getSize()?) Can this be done here without a redundant wrapper, or is getSize really necessary for the API? Thanks!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it would be necessary, should I remove it then?