Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

bogomya
Copy link
Contributor

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
ContributorAuthor

bogomya commentedOct 16, 2018
edited
Loading

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

@itsmichaeldiegoitsmichaeldiego 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

@itsmichaeldiegoitsmichaeldiego merged commit4a92a42 intogoogle-map-react:masterNov 8, 2018
itsmichaeldiego added a commit that referenced this pull requestNov 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
@itsmichaeldiegoitsmichaeldiego mentioned this pull requestNov 8, 2018
itsmichaeldiego added a commit that referenced this pull requestNov 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 version1.1.0, please try it and let me know

DonovanDeSmedt added a commit to DonovanDeSmedt/google-map-react that referenced this pull requestNov 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

lockbot commentedDec 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.

@locklockbot locked asresolvedand limited conversation to collaboratorsDec 1, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers

@itsmichaeldiegoitsmichaeldiegoitsmichaeldiego approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@bogomya@itsmichaeldiego

[8]ページ先頭

©2009-2025 Movatter.jp