Skip to content

call zoomControlClickTime on all clicks#211

Merged
itsmichaeldiego merged 3 commits intogoogle-map-react:masterfrom
alex-e-leon:zoom-on-click
Mar 10, 2018
Merged

call zoomControlClickTime on all clicks#211
itsmichaeldiego merged 3 commits intogoogle-map-react:masterfrom
alex-e-leon:zoom-on-click

Conversation

@alex-e-leon
Copy link
Contributor

fixes an issue where zoomControlClickTime doesn't get called on any clicks not handled by google maps, so if a marker click function should change the zoom level you receive the GoogleMap bounds not eq: error.

@istarkov
Copy link
Collaborator

istarkov commented Aug 2, 2016

Thank you!,
Does it visually changes something? The problem was that in chrome zoom had some bad visual effects, and this partially solved that behavior.
Now I don't remember the idea why I move that code line into if but think that there were some idea, will check in few days.
Also looks like GoogleMap bounds not eq: warning should be deleted (or guarded with some if) as in most cases this messages are useless.

@alex-e-leon
Copy link
Contributor Author

this seems to look ok in chrome.

@istarkov
Copy link
Collaborator

istarkov commented Aug 3, 2016

I need to check this with some of our production code, which causes an addition of double raf hack, it will take some time.

@itsmichaeldiego
Copy link
Member

@alex-e-leon Thanks for your contribution. We would like to have this change but it is having conflicts. Would you mind to add a commit to fix it? Thanks.

@alex-e-leon
Copy link
Contributor Author

@itsmichaeldiego updated

@itsmichaeldiego
Copy link
Member

@alex-e-leon Thanks, I am still a little bit worried about what @istarkov mentioned earlier. As this fix was supposed to improve chrome's visual behavior. Would you mind sharing a test-video? Otherwise give me some time and I will test it thoroughly. Thanks in advance.

@alex-e-leon
Copy link
Contributor Author

No problem @itsmichaeldiego I ended up working around the bug, and I haven't worked on this problem in a while, so I don't have any video I can share. I also don't mind if you close it tbh, but if you feel like working on it let me know how you go!

@itsmichaeldiego
Copy link
Member

@alex-e-leon Oh Alex, thanks and no worries. I will take a look and test it as soon as I have some spare time. Thanks for fixing the conflicts!

@itsmichaeldiego
Copy link
Member

@alex-e-leon I will merge this soon for next release.

@itsmichaeldiego
Copy link
Member

@alex-e-leon I am going to merge this soon, sorry for the delay!

@itsmichaeldiego itsmichaeldiego merged commit e4fa5ff into google-map-react:master Mar 10, 2018
@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.

3 participants

Comments