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

markerClick not work #29

Closed
havo07 opened this issue May 14, 2019 · 14 comments
Closed

markerClick not work #29

havo07 opened this issue May 14, 2019 · 14 comments

Comments

@havo07
Copy link
Collaborator

havo07 commented May 14, 2019

Hi, I'm implementing your npm but when I click on the overlay it gives me the following error, I thought it had to do with "agm-marker-cluster" but it's not like that, to get it I have to perform the click and a small drag

Error is:
Ignored attempt to cancel a touchmove event with cancelable=false, for example because scrolling is in progress and cannot be interrupted.

My code html:
code

Result before click:
result_1

@AckerApple
Copy link
Owner

This package implements a js library called "js-marker-clusterer"

It maybe your issue is coming from there. This package boils down to one component and then it relies on js-marker-clusterer to do the rest.

Link to really only file used in this package:
https://github.com/AckerApple/agm-overlays/blob/master/src/AgmOverlay.component.ts

Link to js-marker-cluster package:
https://www.npmjs.com/package/js-marker-clusterer

@havo07
Copy link
Collaborator Author

havo07 commented May 14, 2019

Remove the agm-marker-cluster code as a test but I can not get an answer, just click and a small drag

@AckerApple
Copy link
Owner

I'm not sure what your last comment is saying?

@havo07
Copy link
Collaborator Author

havo07 commented May 14, 2019

I'm not sure what your last comment is saying?

Sorry.

It works when I do 2 clicks but it expands on the map, when I click and drag

@AckerApple
Copy link
Owner

I think you are just reciting the issue you've already explained.

I went to the demo page for this package on my iPhone. I only had to tap once on a cluster and it zoomed in and separated the cluster into individual markers.

As far as I can tell everything is working as expected.

Demo page:
https://ackerapple.github.io/agm-overlays/

Demo page source code:
https://github.com/AckerApple/agm-overlays/tree/master/example/src

@havo07
Copy link
Collaborator Author

havo07 commented May 14, 2019

The problem is not the cluster, the problem is related to the overlay, the MarkerClick function is not entered, it seems that the MapClick event is by default and not the overlay

I put a video I hope you understand me
https://vimeo.com/user98515389/review/336169451/e9af3dc45a

@AckerApple
Copy link
Owner

The video link does not seem to work.

If I understand you, you want to add a click action to the cluster?

Hey listen, let me save you a lot of time ====> I lost my job just as of Friday. I have no time to investigate or make fixes. You will be on your own. I can accept pull requests at best.

@havo07
Copy link
Collaborator Author

havo07 commented May 14, 2019

You do not understand me, the MapClick event of agm-map is above, for the MarkerClick agm-overlay event, is there any way to put the agm-overlar over the Click event of the map?

@havo07
Copy link
Collaborator Author

havo07 commented May 14, 2019

The video link does not seem to work.

If I understand you, you want to add a click action to the cluster?

Hey listen, let me save you a lot of time ====> I lost my job just as of Friday. I have no time to investigate or make fixes. You will be on your own. I can accept pull requests at best.

Understand, thank you...

@havo07
Copy link
Collaborator Author

havo07 commented May 14, 2019

The video link does not seem to work.

If I understand you, you want to add a click action to the cluster?

Hey listen, let me save you a lot of time ====> I lost my job just as of Friday. I have no time to investigate or make fixes. You will be on your own. I can accept pull requests at best.

I detect a problem with set event "Click" in overlay and previus set event to "Click" in map, the sequence of execute in event is overlay and next steep is map, the fix is modified the file AgmOverlay.component.js, i would like to allow you to collaborate with your repository with this solution.

@AckerApple
Copy link
Owner

Yes I’d accept a pull request of yours. Let me know or look up if you need help on the subject of pull requests. Just fork my code, make the fix, and request a pull request. I’ll even test it.

This package has just a few branches. Be sure to be on the one with the “src” folder and use the scripts in package.json to build and watch

@havo07
Copy link
Collaborator Author

havo07 commented May 15, 2019

Error, remote Permission to summit

@AckerApple
Copy link
Owner

I went to your repositories and you didn't even fork the code for this library.

Please learn how to use the github public system of forking code and then issuing a pull request.

When you do this right, you will have a "fork" of my code here in your repositories: https://github.com/havo07?tab=repositories

Here is the top result for "my first pull request" on youtube:
https://www.youtube.com/watch?v=YTbRzhQju4c

@AckerApple
Copy link
Owner

I published version 1.4.0

If this does not work, try a pull request again but only change what needs to be changed

@havo07 havo07 closed this as completed May 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants