From b12d8eba1df86803ec3c71ba4cf73f6aaeeb51c0 Mon Sep 17 00:00:00 2001
From: Vikram Reddy <gvreddy04@gmail.com>
Date: Wed, 21 Aug 2024 21:52:40 +0530
Subject: [PATCH] GoogleMap component code cleanup

---
 blazorbootstrap/Components/Maps/GoogleMap.razor  |  2 +-
 .../ScriptLoader/ScriptLoader.razor.cs           | 16 ++++++++++++++--
 blazorbootstrap/wwwroot/blazor.bootstrap.js      |  2 +-
 3 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/blazorbootstrap/Components/Maps/GoogleMap.razor b/blazorbootstrap/Components/Maps/GoogleMap.razor
index e818d82f3..f61c6d7e3 100644
--- a/blazorbootstrap/Components/Maps/GoogleMap.razor
+++ b/blazorbootstrap/Components/Maps/GoogleMap.razor
@@ -8,5 +8,5 @@
               Source="@GoogleMapsJsFileUrl"
               OnLoad="OnScriptLoad" />
 
-<div @ref="@Element" id="@Id" class="@ClassNames" style="@StyleNames" role="alert" @attributes="@AdditionalAttributes">
+<div @ref="@Element" id="@Id" class="@ClassNames" style="@StyleNames" @attributes="@AdditionalAttributes">
 </div>
diff --git a/blazorbootstrap/Components/ScriptLoader/ScriptLoader.razor.cs b/blazorbootstrap/Components/ScriptLoader/ScriptLoader.razor.cs
index c6e68b430..d09d3b859 100644
--- a/blazorbootstrap/Components/ScriptLoader/ScriptLoader.razor.cs
+++ b/blazorbootstrap/Components/ScriptLoader/ScriptLoader.razor.cs
@@ -10,7 +10,7 @@ public partial class ScriptLoader : BlazorBootstrapComponentBase
     /// <summary>
     /// The default content type for scripts.
     /// </summary>
-    private const string type = "text/javascript";
+    private const string ScriptType = "text/javascript";
 
     /// <summary>
     /// A reference to this component instance for use in JavaScript calls.
@@ -24,7 +24,7 @@ public partial class ScriptLoader : BlazorBootstrapComponentBase
     protected override async Task OnAfterRenderAsync(bool firstRender)
     {
         if (firstRender)
-            await JSRuntime.InvokeVoidAsync("window.blazorBootstrap.scriptLoader.initialize", Id, Async, Defer, ScriptId, Source, type, objRef);
+            await JSRuntime.InvokeVoidAsync("window.blazorBootstrap.scriptLoader.initialize", Id, Async, Defer, ScriptId, Source, ScriptType, objRef);
 
         await base.OnAfterRenderAsync(firstRender);
     }
@@ -72,6 +72,9 @@ public void OnLoadJS()
     /// <summary>
     /// Gets or sets a value indicating whether the script should be loaded asynchronously.
     /// </summary>
+    /// <remarks>
+    /// Default value is <see langword="false" />.
+    /// </remarks>
     [Parameter]
     public bool Async { get; set; }
 
@@ -79,6 +82,9 @@ public void OnLoadJS()
     /// Gets or sets a value indicating whether the script is meant to be executed 
     /// after the document has been parsed, but before firing DOMContentLoaded event..
     /// </summary>
+    /// <remarks>
+    /// Default value is <see langword="false" />.
+    /// </remarks>
     [Parameter]
     public bool Defer { get; set; }
 
@@ -97,12 +103,18 @@ public void OnLoadJS()
     /// <summary>
     /// Gets or sets the ID of the script element.
     /// </summary>
+    /// <remarks>
+    /// Default value is <see langword="null" />.
+    /// </remarks>
     [Parameter]
     public string? ScriptId { get; set; }
 
     /// <summary>
     /// Gets or sets the URI of the external script to load.
     /// </summary>
+    /// <remarks>
+    /// Default value is <see langword="null" />.
+    /// </remarks>
     [Parameter]
     [EditorRequired]
     public string? Source { get; set; } = default!;
diff --git a/blazorbootstrap/wwwroot/blazor.bootstrap.js b/blazorbootstrap/wwwroot/blazor.bootstrap.js
index 078ebcf44..2733fc87f 100644
--- a/blazorbootstrap/wwwroot/blazor.bootstrap.js
+++ b/blazorbootstrap/wwwroot/blazor.bootstrap.js
@@ -462,7 +462,7 @@ window.blazorBootstrap = {
             return window.blazorBootstrap.googlemaps.instances[elementId];
         },
         initialize: (elementId, zoom, center, markers, clickable, dotNetHelper) => {
-            window.blazorBootstrap.googlemaps.markerEls[elementId] = window.blazorBootstrap.googlemaps.markerEls[elementId] ?? []
+            window.blazorBootstrap.googlemaps.markerEls[elementId] = window.blazorBootstrap.googlemaps.markerEls[elementId] ?? [];
 
             let mapOptions = { center: center, zoom: zoom, mapId: elementId };
             let map = new google.maps.Map(document.getElementById(elementId), mapOptions);