From f02df89d156eb5ee056d2e506d1011749cd7af6e Mon Sep 17 00:00:00 2001
From: "Joseph.Rawlings" <root0091@gmail.com>
Date: Mon, 1 Dec 2025 13:10:49 -0500
Subject: [PATCH] feat: add error message display to Proxmox firewall view for
 improved user feedback and error handling

---
 .../Servers/Proxmox/views/firewall.blade.php  | 23 +++++++++++++++----
 1 file changed, 18 insertions(+), 5 deletions(-)

diff --git a/extensions/Servers/Proxmox/views/firewall.blade.php b/extensions/Servers/Proxmox/views/firewall.blade.php
index d3aacb106..45eed94f0 100644
--- a/extensions/Servers/Proxmox/views/firewall.blade.php
+++ b/extensions/Servers/Proxmox/views/firewall.blade.php
@@ -51,6 +51,24 @@
 </style>
 
 <div class="space-y-6">
+    <!-- Error Message - Top of Page -->
+    @if($error)
+        <div class="bg-error/20 border-2 border-error rounded-lg p-4 shadow-lg">
+            <div class="flex items-start gap-3">
+                <x-ri-error-warning-fill class="size-5 text-error flex-shrink-0 mt-0.5" />
+                <div class="flex-1">
+                    <h3 class="text-base font-semibold text-error mb-1">Firewall Error</h3>
+                    <p class="text-sm text-error/90 leading-relaxed">{{ $error }}</p>
+                </div>
+                <button 
+                    wire:click="$set('error', '')"
+                    class="text-error/70 hover:text-error transition-colors flex-shrink-0">
+                    <x-ri-close-line class="size-5" />
+                </button>
+            </div>
+        </div>
+    @endif
+
     <!-- Firewall Status Card -->
     <div class="bg-background border border-neutral rounded-lg p-6">
         <div class="flex items-center justify-between mb-4">
@@ -76,11 +94,6 @@
                 <span class="text-sm font-medium text-base/70">Enabled</span>
             </div>
         </div>
-        @if($error)
-            <div class="bg-error/10 border border-error/30 rounded-lg p-3 mt-4">
-                <p class="text-sm text-error">{{ $error }}</p>
-            </div>
-        @endif
     </div>
 
     @if($firewallEnabled)