Fix #3885: only re-render the component when message changed.

Oqtane.Client/Modules/Controls/ModuleMessage.razor

@@ -12,12 +12,13 @@
             {
                 <NavLink class="ms-2" href="@NavigateUrl("admin/log")">View Details</NavLink>
             }
-            <a href="@NavigationManager.Uri" class="btn-close" data-dismiss="alert" aria-label="close"></a>
+            <button type="button" class="btn-close" data-dismiss="alert" aria-label="close" @onclick="CloseMessage"></button>
         }
     </div>
 }
 
 @code {
+    private string _message = string.Empty;
     private string _classname = string.Empty;
     private string _formname = "ModuleMessageForm";
 
@@ -27,15 +28,8 @@
     [Parameter]
     public MessageType Type { get; set; }
 
-    public void RefreshMessage(string message, MessageType type)
+    [Parameter]
-    {
+    public RenderModeBoundary Parent { get; set; }
-        Message = message;
-        Type = type;
-
-        UpdateClassName();
-
-        StateHasChanged();
-    }
 
     protected override void OnInitialized()
     {
@@ -47,12 +41,8 @@
 
     protected override void OnParametersSet()
     {
-        UpdateClassName();
+        _message = Message;
-    }
+        if (!string.IsNullOrEmpty(_message))
-
-    private void UpdateClassName()
-    {
-        if (!string.IsNullOrEmpty(Message))
         {
             _classname = GetMessageType(Type);
         }
@@ -79,4 +69,15 @@
 
         return classname;
     }
+    private void CloseMessage(MouseEventArgs e)
+    {
+        if(Parent != null)
+        {
+            Parent.DismissMessage();
+        }
+        else
+        {
+            NavigationManager.NavigateTo(NavigationManager.Uri);
+        }
+    }
 }

Oqtane.Client/Modules/ModuleBase.cs

@@ -278,7 +278,6 @@ namespace Oqtane.Modules
 
         public void AddModuleMessage(string message, MessageType type, string position)
         {
-            ClearModuleMessage();
             RenderModeBoundary.AddModuleMessage(message, type, position);
         }
 

Oqtane.Client/UI/RenderModeBoundary.razor

@@ -10,13 +10,19 @@
         {
             @if (ModuleType != null)
             {
-                <ModuleMessage @ref="moduleMessageTop" Message="@_messageContent" Type="@_messageType" />
+                @if (!string.IsNullOrEmpty(_messageContent) && _messagePosition == "top")
+                {
+                    <ModuleMessage Message="@_messageContent" Type="@_messageType" Parent="@this" />
+                }
                 @DynamicComponent
                 @if (_progressIndicator)
                 {
                     <div class="app-progress-indicator"></div>
                 }
-                <ModuleMessage @ref="moduleMessageBottom" Message="@_messageContent" Type="@_messageType" />
+                @if (!string.IsNullOrEmpty(_messageContent) && _messagePosition == "bottom")
+                {
+                    <ModuleMessage Message="@_messageContent" Type="@_messageType" Parent="@this" />
+                }
             }
         }
         else
@@ -41,8 +47,6 @@
     private string _messagePosition;
     private bool _progressIndicator = false;
     private string _error;
-    private ModuleMessage moduleMessageTop;
-    private ModuleMessage moduleMessageBottom;
 
     [Parameter]
     public SiteState SiteState { get; set; }
@@ -102,13 +106,18 @@
     }
 
     public void AddModuleMessage(string message, MessageType type, string position)
+    {
+        if(message != _messageContent
+            || type != _messageType
+            || position != _messagePosition)
         {
             _messageContent = message;
             _messageType = type;
             _messagePosition = position;
             _progressIndicator = false;
 
-        Refresh();
+            StateHasChanged();
+        }
     }
 
     public void ShowProgressIndicator()
@@ -123,25 +132,10 @@
         StateHasChanged();
     }
 
-    private void DismissMessage()
+    public void DismissMessage()
     {
         _messageContent = "";
-    }
+        StateHasChanged();
-
-    private void Refresh()
-    {
-        var updateTop = string.IsNullOrEmpty(_messageContent) || _messagePosition == "top";
-        var updateBottom = string.IsNullOrEmpty(_messageContent) || _messagePosition == "bottom";
-
-        if (updateTop && moduleMessageTop != null)
-        {
-            moduleMessageTop.RefreshMessage(_messageContent, _messageType);
-        }
-
-        if (updateBottom && moduleMessageBottom != null)
-        {
-            moduleMessageBottom.RefreshMessage(_messageContent, _messageType);
-        }
     }
 
     protected override async Task OnErrorAsync(Exception exception)