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

DrawingManager.SetDrawingMode no longer works in 3.3.2 and newer #320

Closed
Mr-Technician opened this issue Mar 13, 2024 · 7 comments
Closed

Comments

@Mr-Technician
Copy link

Mr-Technician commented Mar 13, 2024

Calling await DrawingManager.SetDrawingMode(OverlayType.Polygon); in versions newer than 3.3.2 has no effect. I suspect this is due to the refactoring in the Helper.cs file that modified how enums are handled. In the meantime, I am using an older version of the library in my project.

I poked around for a while but wasn't able to pinpoint the specific issue. This commit may be the culprit: 4af3864

But there was also this commit later, so I am not sure what to make of it: 32ffc89

Edit: this issue can be replicated in the ServerSideDemo on the /drawing-manager page. Clicking "Change mode to line" does nothing.

@valentasm1
Copy link
Collaborator

Sorry but i will take a look next week. Tuesday.

@Mr-Technician
Copy link
Author

Did you have a chance to look at this?

@valentasm1
Copy link
Collaborator

Fixed. Thank you for detailed information
https://www.nuget.org/packages/BlazorGoogleMaps/4.1.1

@Mr-Technician
Copy link
Author

Thank you!

@Mr-Technician
Copy link
Author

Mr-Technician commented Mar 21, 2024

@valentasm1 I am still having an issue with the Polygon overlay. My button that switches to Polygon mode doesn't work unless I first use my Polyline button. Similarly, the ChangeDrawingModeToLine on the drawing manager demo page works, but if I manually update the method as follows:

private async Task ChangeDrawingModeToLine()
{
    await _drawingManager.SetDrawingMode(OverlayType.Polygon);
}

And then run the project, the Change Mode to Line button no longer works.

@valentasm1
Copy link
Collaborator

@Mr-Technician
Copy link
Author

Thank you very much!

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

2 participants