From 7b88610f155758ae9f160211c9d7d3f0f302ad15 Mon Sep 17 00:00:00 2001
From: "Joseph.Rawlings" <root0091@gmail.com>
Date: Thu, 4 Dec 2025 10:26:13 -0500
Subject: [PATCH] style: update Proxmox firewall view layout by removing
 unnecessary IDs and enhancing card styles for improved visual consistency and
 performance

---
 .../Proxmox/views/components/layout.blade.php |  18 +--
 .../Servers/Proxmox/views/firewall.blade.php  | 140 +-----------------
 2 files changed, 17 insertions(+), 141 deletions(-)

diff --git a/extensions/Servers/Proxmox/views/components/layout.blade.php b/extensions/Servers/Proxmox/views/components/layout.blade.php
index a517fd135..d0cd878b6 100644
--- a/extensions/Servers/Proxmox/views/components/layout.blade.php
+++ b/extensions/Servers/Proxmox/views/components/layout.blade.php
@@ -443,23 +443,23 @@
                         left: 50%;
                         transform: translateX(-50%);
                         min-width: 180px;
-                        background: color-mix(in srgb, var(--color-background-secondary) 60%, transparent);
-                        backdrop-filter: blur(12px);
-                        border: 1px solid color-mix(in srgb, var(--color-neutral) 30%, transparent);
+                        background: color-mix(in srgb, var(--color-background-secondary) 85%, transparent);
+                        backdrop-filter: blur(16px);
+                        border: 1px solid color-mix(in srgb, var(--color-neutral) 40%, transparent);
                         border-radius: 12px;
                         box-shadow: 
-                            0 4px 16px rgba(0, 0, 0, 0.12),
-                            0 0 0 1px rgba(0, 0, 0, 0.03);
+                            0 4px 16px rgba(0, 0, 0, 0.15),
+                            0 0 0 1px rgba(0, 0, 0, 0.05);
                         z-index: 50;
                         overflow: hidden;
                     }
 
                     .dark .dropdown-menu {
-                        background: color-mix(in srgb, var(--color-background-secondary) 80%, transparent);
-                        border-color: color-mix(in srgb, var(--color-neutral) 40%, transparent);
+                        background: color-mix(in srgb, var(--color-background-secondary) 92%, transparent);
+                        border-color: color-mix(in srgb, var(--color-neutral) 50%, transparent);
                         box-shadow: 
-                            0 4px 20px rgba(0, 0, 0, 0.4),
-                            0 0 0 1px rgba(255, 255, 255, 0.08);
+                            0 4px 20px rgba(0, 0, 0, 0.45),
+                            0 0 0 1px rgba(255, 255, 255, 0.1);
                     }
 
                     .dropdown-item {
diff --git a/extensions/Servers/Proxmox/views/firewall.blade.php b/extensions/Servers/Proxmox/views/firewall.blade.php
index 39e99b531..cc5ccb418 100644
--- a/extensions/Servers/Proxmox/views/firewall.blade.php
+++ b/extensions/Servers/Proxmox/views/firewall.blade.php
@@ -371,132 +371,9 @@
     }
 </style>
 
-<div x-data="{
-    activeSection: 'status',
-    scrollToSection(sectionId) {
-        this.activeSection = sectionId;
-        const element = document.getElementById(sectionId);
-        if (element) {
-            element.scrollIntoView({ behavior: 'smooth', block: 'start' });
-        }
-    },
-    init() {
-        // Set up intersection observer to update active section on scroll
-        const observer = new IntersectionObserver((entries) => {
-            entries.forEach(entry => {
-                if (entry.isIntersecting && entry.intersectionRatio > 0.2) {
-                    this.activeSection = entry.target.id;
-                }
-            });
-        }, { threshold: 0.2, rootMargin: '-10% 0px -70% 0px' });
-        
-        // Observe all sections
-        document.querySelectorAll('.firewall-section-card').forEach(section => {
-            observer.observe(section);
-        });
-    }
-}" class="lg:grid lg:grid-cols-5 gap-6">
-    
-    <!-- Sidebar Navigation -->
-    <div class="lg:col-span-1 mb-6 lg:mb-0">
-        <div class="firewall-sidebar bg-background-secondary border border-neutral rounded-lg p-4">
-            <div class="flex items-center gap-2 mb-4 pb-3 border-b border-neutral/30">
-                <x-ri-shield-line class="size-5 text-primary" />
-                <h3 class="text-base font-semibold">Firewall</h3>
-            </div>
-            
-            <nav class="space-y-1">
-                <!-- Status Section -->
-                <button 
-                    @click="scrollToSection('status')"
-                    :class="activeSection === 'status' ? 'active' : ''"
-                    class="firewall-nav-item">
-                    <x-ri-toggle-line class="firewall-nav-icon" />
-                    <span>Status</span>
-                    @if($firewallEnabled)
-                        <span class="firewall-nav-badge firewall-nav-badge-success">ON</span>
-                    @else
-                        <span class="firewall-nav-badge firewall-nav-badge-neutral">OFF</span>
-                    @endif
-                </button>
-                
-                <!-- Network Adapters Section -->
-                <button 
-                    @click="scrollToSection('adapters')"
-                    :class="activeSection === 'adapters' ? 'active' : ''"
-                    class="firewall-nav-item">
-                    <x-ri-wifi-line class="firewall-nav-icon" />
-                    <span>Adapters</span>
-                    <span class="firewall-nav-badge firewall-nav-badge-neutral">{{ count($networkAdapters) }}</span>
-                </button>
-                
-                @if($firewallEnabled)
-                    <!-- Firewall Rules Section -->
-                    <button 
-                        @click="scrollToSection('rules')"
-                        :class="activeSection === 'rules' ? 'active' : ''"
-                        class="firewall-nav-item">
-                        <x-ri-list-check class="firewall-nav-icon" />
-                        <span>Rules</span>
-                        <span class="firewall-nav-badge firewall-nav-badge-neutral">{{ count($rules) }}</span>
-                    </button>
-                    
-                    <!-- Firewall Logs Section -->
-                    <button 
-                        @click="scrollToSection('logs')"
-                        :class="activeSection === 'logs' ? 'active' : ''"
-                        class="firewall-nav-item">
-                        <x-ri-file-list-3-line class="firewall-nav-icon" />
-                        <span>Logs</span>
-                        @if(count($firewallLogs) > 0)
-                            <span class="firewall-nav-badge firewall-nav-badge-warning">{{ count($firewallLogs) }}</span>
-                        @else
-                            <span class="firewall-nav-badge firewall-nav-badge-neutral">0</span>
-                        @endif
-                    </button>
-                @endif
-            </nav>
-            
-            <!-- Quick Actions -->
-            <div class="mt-4 pt-4 border-t border-neutral/30">
-                <p class="text-xs font-semibold text-base/50 uppercase tracking-wide mb-2">Quick Actions</p>
-                <div class="space-y-1">
-                    <button 
-                        wire:click="toggleFirewall"
-                        wire:loading.attr="disabled"
-                        class="firewall-nav-item text-sm {{ $firewallEnabled ? 'text-error/80 hover:text-error' : 'text-success/80 hover:text-success' }}">
-                        @if($firewallEnabled)
-                            <x-ri-shield-cross-line class="firewall-nav-icon" />
-                            <span>Disable Firewall</span>
-                        @else
-                            <x-ri-shield-check-line class="firewall-nav-icon" />
-                            <span>Enable Firewall</span>
-                        @endif
-                    </button>
-                    @if($firewallEnabled)
-                        <button 
-                            wire:click="startCreateRule"
-                            class="firewall-nav-item text-sm text-primary/80 hover:text-primary">
-                            <x-ri-add-line class="firewall-nav-icon" />
-                            <span>Add Rule</span>
-                        </button>
-                        <button 
-                            wire:click="loadFirewallLogs"
-                            wire:loading.attr="disabled"
-                            class="firewall-nav-item text-sm">
-                            <x-ri-refresh-line class="firewall-nav-icon" />
-                            <span>Refresh Logs</span>
-                        </button>
-                    @endif
-                </div>
-            </div>
-        </div>
-    </div>
-    
-    <!-- Main Content -->
-    <div class="lg:col-span-4 space-y-6">
-        <!-- Error Message - Centered Modal -->
-        @if($error && $showErrorModal)
+<div class="space-y-6">
+    <!-- Error Message - Centered Modal -->
+    @if($error && $showErrorModal)
         <div class="fixed inset-0 flex items-center justify-center z-50 bg-black/50 backdrop-blur-sm" 
              wire:click="$set('error', ''); $set('showErrorModal', false)">
             <div class="bg-error/20 border-2 border-error rounded-lg p-6 shadow-lg max-w-3xl w-full mx-4 max-h-[80vh] overflow-y-auto" wire:click.stop>
@@ -519,7 +396,7 @@
     @endif
 
     <!-- Firewall Status Card -->
-    <div id="status" class="firewall-section-card bg-background border border-neutral rounded-lg p-6">
+    <div class="bg-background border border-neutral rounded-lg p-6">
         <div class="flex items-center justify-between mb-4">
             <div class="flex items-center gap-3">
                 <div class="flex items-center justify-center w-10 h-10 rounded-lg bg-primary/10">
@@ -546,7 +423,7 @@
     </div>
 
     <!-- Network Adapters Card -->
-    <div id="adapters" class="firewall-section-card bg-background border border-neutral rounded-lg p-6">
+    <div class="bg-background border border-neutral rounded-lg p-6">
         <div class="flex items-center gap-3 mb-4">
             <div class="flex items-center justify-center w-10 h-10 rounded-lg bg-primary/10">
                 <x-ri-wifi-line class="size-5 text-primary" />
@@ -605,7 +482,7 @@
 
     @if($firewallEnabled)
         <!-- Firewall Rules Card -->
-        <div id="rules" class="firewall-section-card bg-background border border-neutral rounded-lg p-6">
+        <div class="bg-background border border-neutral rounded-lg p-6">
             <div class="flex items-center justify-between mb-4">
                 <div class="flex items-center gap-3">
                     <div class="flex items-center justify-center w-10 h-10 rounded-lg bg-primary/10">
@@ -705,7 +582,7 @@
         </div>
 
         <!-- Firewall Logs Card -->
-        <div id="logs" class="firewall-section-card bg-background border border-neutral rounded-lg p-6">
+        <div class="bg-background border border-neutral rounded-lg p-6">
             <div class="flex items-center justify-between mb-4">
                 <div class="flex items-center gap-3">
                     <div class="flex items-center justify-center w-10 h-10 rounded-lg bg-primary/10">
@@ -2033,8 +1910,7 @@
             </div>
         </div>
     @endif
-    </div><!-- End Main Content (lg:col-span-4) -->
-</div><!-- End Grid Container -->
+</div>
 
 @script
 <script>