Thanks to visit codestin.com
Credit goes to github.com

Skip to content

call zoomControlClickTime on all clicks - #211

Merged
itsmichaeldiego merged 3 commits into
GoogleMapReact:masterfrom
alex-e-leon:zoom-on-click
Mar 10, 2018
Merged

itsmichaeldiego merged 3 commits into
GoogleMapReact:masterfrom
alex-e-leon:zoom-on-click

Conversation

@alex-e-leon

Copy link
Copy Markdown
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

istarkov commented Aug 2, 2016

Copy link
Copy Markdown
Member

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
Copy Markdown
Contributor Author

this seems to look ok in chrome.

@istarkov

istarkov commented Aug 3, 2016

Copy link
Copy Markdown
Member

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
Copy Markdown
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
Copy Markdown
Contributor Author

@itsmichaeldiego updated

@itsmichaeldiego

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

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

@itsmichaeldiego

Copy link
Copy Markdown
Member

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

@itsmichaeldiego
itsmichaeldiego merged commit e4fa5ff into GoogleMapReact:master Mar 10, 2018
@lock

lock Bot commented Dec 1, 2019

Copy link
Copy Markdown

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