-
Notifications
You must be signed in to change notification settings - Fork 52
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
Bug: GvmInfoWindow components are not removed from the map #323
Comments
The inject key can only be used on descendants not in siblings. Composables on mounted hook are more accurate. Solve: #323
* fix(v3): change inject for composable to get component promise The inject key can only be used on descendants not in siblings. Composables on mounted hook are more accurate. Solve: #323 * test(v3): update cluster test to use the new key feature * chore(root): update package-manager y package.json and gh workflows
@Tuhtarov thank you again. I found a logical issue here, and now it's working as expected. I'm closing the issues considering it solved, feel free to re-open it if needed or open a new one. |
And thank you again :) |
@diegoazh, hello again! |
Hi @Tuhtarov I checked and it works as expected, you can check it in this stackblitz example.
Please check the implementation on the |
Hi @diegoazh, I appreciate your work and look forward to your response. |
Describe the bug
GvmInfoWindow components are not deleted (not closed) in a vue template when the object responsible for rendering this component is deleted in a reactive variable.
To reproduce
For example, we have a display of 3 markers and 3 windows above them. After deleting any element from the storage, the marker and the window must be deleted from map. Currently, only the marker is deleted. But the window remains on the map.
Demo code roughly describing my component for displaying some data on a map:
Expected behavior
The marker and window should be removed from the map if the element responsible for drawing the marker and window was deleted in the data source.
Current behavior
Only the marker is removed, the window remains.
Additional context
Using this crutch code, you can close these windows on the map
Screenshots
Desktop
Versions
Package manager
Plugin version
The text was updated successfully, but these errors were encountered: