serdarciplak / BlazorMonaco

Blazor component for Microsoft's Monaco Editor which powers Visual Studio Code.
https://serdarciplak.github.io/BlazorMonaco/
MIT License
441 stars 99 forks source link

BlazorMonaco.Environment overlap with System.Environment #73

Closed svenboulanger closed 1 year ago

svenboulanger commented 1 year ago

When including BlazorMonaco and System which would happen quite commonly, there is always an ambiguous reference to Environment (like when using Environment.Newline).

More of a user-experience issue than a bug though. People can simply qualify the full namespace where necessary.

serdarciplak commented 1 year ago

As the BlazorMonaco.Environment class is not currently used, I can remove it until it's used. But that will be just postponing the issue. Or, another similar name overlap may occur in any other place with any other namespace. I think the best way to handle this issue is using the full class name to get rid of the ambiguity, like you did.