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

Was onClucterClick removed as an event? #142

Open
GeorgeCodeHub opened this issue Apr 13, 2021 · 5 comments
Open

Was onClucterClick removed as an event? #142

GeorgeCodeHub opened this issue Apr 13, 2021 · 5 comments

Comments

@GeorgeCodeHub
Copy link

GeorgeCodeHub commented Apr 13, 2021

Hello there. I'm trying to do an activity where I click on a cluster and something happens. However, I am unable to trigger it on the cluster. Below an example on codesandbox. Try clicking on the cluster. You will see that nothing happens. Was it removed?

https://codesandbox.io/s/gifted-liskov-9rw91?file=/src/App.js

@charlesomer
Copy link

charlesomer commented May 8, 2021

I think I'm having the same problem, perhaps an issue with v3?

@charlesomer
Copy link

Looks like the storybook examples don't work either unfortunately apart from the onClose one.

@politzekv1
Copy link

The thing with this component implementation is that createPathComponent needs 2 arguments. The first is the function that called when the component is created, and the second is a function that called when the component is updated.
And this implementation defines only the first one.

({ children: _c, ...props }, ctx) => {

So if you dynamically add/remove handlers to your component, changes are not applied.

@stale
Copy link

stale bot commented Jun 30, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Jun 30, 2021
@yuzhva yuzhva added enhancement and removed wontfix labels Jul 6, 2021
@lgrayland
Copy link

was also looking for an answer to this but seems i have found a solution, just call onClick not onClusterClick.

this line is causing the clustering event to be called clusterclusterclick, not sure if there was a need to ad this extra text for other use cases or can be removed.

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

5 participants