Skip to content

Pass map instance to onDrag handler #656

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

Merged
merged 1 commit into from
Nov 8, 2018

Conversation

bogomya
Copy link
Contributor

@bogomya bogomya commented Oct 11, 2018

Goal:
Add possibility to work with google map api in the onDrag handler.

  • documentation improving

@itsmichaeldiego
Copy link
Member

@bogomya Could you explain how this improves and provide an example? Just for documentation, I totally understand what you are doing

@bogomya
Copy link
Contributor Author

bogomya commented Oct 16, 2018

The idea was in getting map bounds on drag event to load new markers. I could use onChange, but onChange is called also when zoom event is occurred, and also on initial load.

@itsmichaeldiego
Copy link
Member

@bogomya Thank you so much! This would be helpful. Could you provide a live example of this working? I need to check its working before releasing it.

Copy link
Member

@itsmichaeldiego itsmichaeldiego left a comment

Choose a reason for hiding this comment

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

Looking forward to see the example

@itsmichaeldiego itsmichaeldiego merged commit 4a92a42 into google-map-react:master Nov 8, 2018
itsmichaeldiego added a commit that referenced this pull request Nov 8, 2018
This includes: 

- #655 -> Add math abs to avoid negative values when calculating zoom 
- #656 -> Pass map instance to onDrag handler 
- #593 -> Added feature: update heat map on data change + fix linting
itsmichaeldiego added a commit that referenced this pull request Nov 8, 2018
This includes: 

- #655 Add math abs to avoid negative values when calculating zoom 
- #656 Pass map instance to onDrag handler 
- #593 Added feature: update heat map on data change + fix linting
@itsmichaeldiego
Copy link
Member

@bogomya Merged and deployed in version 1.1.0, please try it and let me know

DonovanDeSmedt added a commit to DonovanDeSmedt/google-map-react that referenced this pull request Nov 21, 2018
* 'master' of github.com:google-map-react/google-map-react:
  Bump to 1.1.1 (google-map-react#680)
  Revert "Added feature: update heat map on data change + fix linting" (google-map-react#679)
  Bump version to 1.1.0 (google-map-react#671)
  Added feature: update heat map on data change + fix linting (google-map-react#593)
  Pass map instance to onDrag handler (google-map-react#656)
  add math abs to avoid negative values when calculating zoom (google-map-react#655)
  Bump version to 1.0.9 (google-map-react#651)
  Custom div style options (google-map-react#634)
  Bump version to 1.0.8 (google-map-react#646)
  Revert 643 fix/map context (google-map-react#645)
  Bump version to 1.0.7 (google-map-react#644)
  Add passive scroll (google-map-react#631)
  Use React 16 portal to render map overlay (google-map-react#643)
  Fix old examples links and add one to new examples (google-map-react#633)
  Bump version to 1.0.6 (google-map-react#621)
  Add prop `onTilesLoaded` (google-map-react#615)
  Fix typo, and call fromContainerPixelToLatLng() as you would expect. (google-map-react#620)
  Update API.md (google-map-react#611)
  Upgrade version to 1.0.5 (google-map-react#607)
  Remove marker jiggle. (google-map-react#603)
@lock
Copy link

lock bot commented Dec 1, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Dec 1, 2019
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