From 9ebd882c3f18d1a1747ac366cc82ba0e9beb8519 Mon Sep 17 00:00:00 2001
From: Ben <zyhfish@163.com>
Date: Tue, 27 Feb 2024 23:08:27 +0800
Subject: [PATCH] remove the visible parameter in module message component.

---
 .../Modules/Controls/ModuleMessage.razor         |  8 ++------
 Oqtane.Client/UI/RenderModeBoundary.razor        | 16 ++++++++--------
 2 files changed, 10 insertions(+), 14 deletions(-)

diff --git a/Oqtane.Client/Modules/Controls/ModuleMessage.razor b/Oqtane.Client/Modules/Controls/ModuleMessage.razor
index b3522e0cf..7771a7a94 100644
--- a/Oqtane.Client/Modules/Controls/ModuleMessage.razor
+++ b/Oqtane.Client/Modules/Controls/ModuleMessage.razor
@@ -2,7 +2,7 @@
 @inherits ModuleControlBase
 @inject NavigationManager NavigationManager
 
-@if (Visible && !string.IsNullOrEmpty(Message))
+@if (!string.IsNullOrEmpty(Message))
 {
     <div class="@_classname alert-dismissible fade show mb-3" role="alert">
         @((MarkupString)Message)
@@ -30,14 +30,10 @@
     [Parameter]
     public MessageType Type { get; set; }
 
-    [Parameter]
-    public bool Visible { get; set; } = true;
-
-    public void RefreshMessage(string message, MessageType type, bool visible)
+    public void RefreshMessage(string message, MessageType type)
     {
         Message = message;
         Type = type;
-        Visible = visible;
 
         UpdateClassName();
 
diff --git a/Oqtane.Client/UI/RenderModeBoundary.razor b/Oqtane.Client/UI/RenderModeBoundary.razor
index 2236e5382..c407031ab 100644
--- a/Oqtane.Client/UI/RenderModeBoundary.razor
+++ b/Oqtane.Client/UI/RenderModeBoundary.razor
@@ -11,13 +11,13 @@
             @if (ModuleType != null)
             {
                 @((MarkupString)$"<!-- rendermode: {ModuleState.RenderMode} -->")
-                <ModuleMessage @ref="moduleMessageTop" Message="@_messageContent" Type="@_messageType" Visible="@(!string.IsNullOrEmpty(_messageContent) && _messagePosition == "top")" />
+                <ModuleMessage @ref="moduleMessageTop" Message="@_messageContent" Type="@_messageType" />
                 @DynamicComponent
                 @if (_progressIndicator)
                 {
                     <div class="app-progress-indicator"></div>
                 }
-                <ModuleMessage @ref="moduleMessageBottom" Message="@_messageContent" Type="@_messageType" Visible="@(!string.IsNullOrEmpty(_messageContent) && _messagePosition == "bottom")" />
+                <ModuleMessage @ref="moduleMessageBottom" Message="@_messageContent" Type="@_messageType" />
             }
         }
         else
@@ -131,17 +131,17 @@
 
     private void Refresh()
     {
-        var messageTopVisible = !string.IsNullOrEmpty(_messageContent) && _messagePosition == "top";
-        var messageBottomVisible = !string.IsNullOrEmpty(_messageContent) && _messagePosition == "bottom";
+        var updateTop = string.IsNullOrEmpty(_messageContent) || _messagePosition == "top";
+        var updateBottom = string.IsNullOrEmpty(_messageContent) || _messagePosition == "bottom";
 
-        if (moduleMessageTop != null)
+        if (updateTop && moduleMessageTop != null)
         {
-            moduleMessageTop.RefreshMessage(_messageContent, _messageType, messageTopVisible);
+            moduleMessageTop.RefreshMessage(_messageContent, _messageType);
         }
 
-        if (moduleMessageBottom != null)
+        if (updateBottom && moduleMessageBottom != null)
         {
-            moduleMessageBottom.RefreshMessage(_messageContent, _messageType, messageBottomVisible);
+            moduleMessageBottom.RefreshMessage(_messageContent, _messageType);
         }
     }