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

Attempt to fix closeclick issue in InfoWindow #52

Merged
merged 1 commit into from
Feb 24, 2023

Conversation

oliverkirsch
Copy link

Propagate google maps InfoWindow's closeclick event to the parent component however, re-open the InfoWindow when the event is received in order to ensure, its state is set solely by the "opened" prop of the InfoWindow component

Do not emit closeclick in the _openInfoWindow() method triggered by the watcher on the opened prop. closeclick should only be triggered, when the user atually clicked close

- propagate google maps InfoWindow's closeclick event to the parent component
  however, re-open the InfoWindow to ensure, its state is set solely by
  the opened prop of the InfoWindow component
- do not emit closeclick in the _openInfoWindow() method triggered by
  the watcher on the opened prop. closeclick should only be triggered,
  when the user atually clicked close
@netlify
Copy link

netlify bot commented Feb 17, 2023

Deploy Preview for vue-google-maps-community-fork canceled.

Name Link
🔨 Latest commit 0ea377c
🔍 Latest deploy log https://app.netlify.com/sites/vue-google-maps-community-fork/deploys/63f398edf0fe0c0007ecf866

@NathanAP
Copy link
Owner

If someone uses this and wants to help us improve it, please let us know. I'll leave it here some days to see if someone appears. Otherwise I'll accept it.

Thanks again for your help!

@NathanAP NathanAP added bug Something isn't working help wanted You can contribute with this with in any way possible waiting We need more info about it or we are waiting for something else labels Feb 20, 2023
@NathanAP NathanAP self-assigned this Feb 20, 2023
@NathanAP NathanAP removed help wanted You can contribute with this with in any way possible waiting We need more info about it or we are waiting for something else labels Feb 24, 2023
@NathanAP NathanAP merged commit f1fcf4b into NathanAP:main Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants