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

Add way to set AdvancedMarker properties #338

Closed
valentasm1 opened this issue Jun 20, 2024 · 6 comments
Closed

Add way to set AdvancedMarker properties #338

valentasm1 opened this issue Jun 20, 2024 · 6 comments

Comments

@valentasm1
Copy link
Collaborator

valentasm1 commented Jun 20, 2024

Currently there are no way to setup property.
Maybe do check if object has such property and set that. Need to check if it has impact on perfomance
One example would be update position for advancedMarker after it is created. Currently there are only properties

@orbital-git
Copy link

Waiting for it.

Thanks

@valentasm1
Copy link
Collaborator Author

Finished for position
https://www.nuget.org/packages/BlazorGoogleMaps/4.6.1

@orbital-git
Copy link

Hi,

Very nice! Works great. Thanks

I was wondering if it is possible to add other properties as well. It would be great if we can change the Content property. For instance, I would like to change the Svg to reflect a new status.

@valentasm1
Copy link
Collaborator Author

Added way to set AdvancedMarker content property
https://www.nuget.org/packages/BlazorGoogleMaps/4.6.2

@Eddie-Hartman
Copy link
Contributor

Eddie-Hartman commented Aug 3, 2024

@valentasm1 I noticed that Content supports OneOf string or PinElement, but the SetContent method does not support PinElement. I would need that functionality.

I also tried to set content to HTML as described here: https://developers.google.com/maps/documentation/javascript/advanced-markers/graphic-markers#icon-fonts and just got it as text. Does it only support image links currently?

Sorry for the flood of issue, request, question. Just getting into using the library.

@valentasm1
Copy link
Collaborator Author

@Eddie-Hartman regarding font you should create separate issue. It could be due to that it creates template instead div.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants