From 55d762cf5aa5755350a5d8fcf72f50d2112b8758 Mon Sep 17 00:00:00 2001
From: "Joseph.Rawlings" <root0091@gmail.com>
Date: Thu, 4 Dec 2025 19:27:26 -0500
Subject: [PATCH] refactor: remove IP location modal from Proxmox firewall view
 to streamline UI and improve performance

---
 .../Servers/Proxmox/views/firewall.blade.php  | 80 -------------------
 1 file changed, 80 deletions(-)

diff --git a/extensions/Servers/Proxmox/views/firewall.blade.php b/extensions/Servers/Proxmox/views/firewall.blade.php
index 64e1f8f80..2fe28ca96 100644
--- a/extensions/Servers/Proxmox/views/firewall.blade.php
+++ b/extensions/Servers/Proxmox/views/firewall.blade.php
@@ -543,86 +543,6 @@
                                                     </div>
                                                     @endif
                                             </div>
-                                        </td>
-                                            <td class="py-3 px-4 text-sm font-mono text-xs">
-                                            <div class="flex items-center gap-1">
-                                                    <span>{{ $rule['dest'] ?? '-' }}</span>
-                                                    @php $ruleDestIP = $rule['dest'] ?? ''; @endphp
-                                                    @if($ruleDestIP && $ruleDestIP !== '-' && $this->checkIsPublicIP($ruleDestIP))
-                                                    <div class="relative" x-data="{ open: false, loading: false, ipData: null }" x-cloak>
-                                                            <button type="button" @click.stop="
-                                                            if (!ipData && !loading) {
-                                                                loading = true;
-                                                                open = true;
-                                                                    window.fetchIPLocation('{{ $ruleDestIP }}').then(data => {
-                                                                    ipData = data;
-                                                                    loading = false;
-                                                                }).catch(err => {
-                                                                        ipData = { status: 'error', message: 'Failed to load' };
-                                                                    loading = false;
-                                                                });
-                                                            } else {
-                                                                open = true;
-                                                            }
-                                                        " class="text-base/50 hover:text-base/70 transition-colors" title="View IP Location">
-                                                            <x-ri-map-pin-line class="size-3" />
-                                                        </button>
-                                                            
-                                                            {{-- IP Location Modal (same structure) --}}
-                                                            <div x-show="open" x-transition:enter="transition ease-out duration-200" x-transition:enter-start="opacity-0" x-transition:enter-end="opacity-100" x-transition:leave="transition ease-in duration-150" x-transition:leave-start="opacity-100" x-transition:leave-end="opacity-0" @click="open = false" @keydown.escape.window="open = false" class="fixed inset-0 z-50 bg-black/50 backdrop-blur-sm flex items-center justify-center p-4" style="display: none;">
-                                                                <div @click.stop class="relative w-full max-w-4xl bg-background-secondary rounded-xl border border-neutral/20 shadow-2xl max-h-[90vh] overflow-y-auto">
-                                                                <button @click="open = false" class="absolute top-4 right-4 text-base/50 hover:text-base transition-colors z-10">
-                                                                    <x-ri-close-line class="size-5" />
-                                                                </button>
-                                                                    <div class="p-6">
-                                                                    <div class="flex items-center gap-2 mb-4">
-                                                                            <x-ri-map-pin-line class="size-5 text-primary" />
-                                                                            <h3 class="text-lg font-semibold">IP Address: <span class="font-mono">{{ $ruleDestIP }}</span></h3>
-                                                                    </div>
-                                                                    <div x-show="loading" class="text-center py-8">
-                                                                        <svg fill="currentColor" stroke="currentColor" viewBox="0 0 128 128" class="size-6 mx-auto mb-2">
-                                                                                <g><path stroke-width="5" d="M31.6 8.98a4.02 4.02 0 0 0 1.7 5.3l6.12 3.22a52.56 52.56 0 1 0 63.13 10.73L96 33.97a43.88 43.88 0 1 1-53-8.5l-3.65 7.4a4.02 4.02 0 0 0 1.72 5.3 3.73 3.73 0 0 0 5.13-1.67l7.66-15.63a4.18 4.18 0 0 0-1.75-5.4L36.74 7.32a3.73 3.73 0 0 0-5.13 1.66z" fill="currentColor" stroke="currentColor"/><animateTransform attributeName="transform" type="rotate" from="0 64 64" to="360 64 64" dur="1000ms" repeatCount="indefinite"></animateTransform></g>
-                                                                        </svg>
-                                                                        <p class="text-sm text-base/60">Loading location data...</p>
-                                                                    </div>
-                                                                    <div x-show="ipData && !loading">
-                                                                            <template x-if="ipData && ipData.status === 'success'">
-                                                                            <div>
-                                                                                <div class="grid grid-cols-1 md:grid-cols-2 gap-4 mb-4">
-                                                                                        <div class="space-y-2">
-                                                                                            <div><span class="text-xs text-base/60">Country:</span> <span class="font-semibold text-sm" x-text="ipData.country"></span></div>
-                                                                                            <div><span class="text-xs text-base/60">Region:</span> <span class="font-semibold text-sm" x-text="ipData.regionName + ' (' + ipData.region + ')'"></span></div>
-                                                                                            <div><span class="text-xs text-base/60">City:</span> <span class="font-semibold text-sm" x-text="ipData.city"></span></div>
-                                                                                            <div><span class="text-xs text-base/60">ZIP:</span> <span class="font-semibold text-sm" x-text="ipData.zip"></span></div>
-                                                                                            <div><span class="text-xs text-base/60">Timezone:</span> <span class="font-semibold text-sm" x-text="ipData.timezone"></span></div>
-                                                                                    </div>
-                                                                                        <div class="space-y-2">
-                                                                                            <div><span class="text-xs text-base/60">ISP:</span> <span class="font-semibold text-sm" x-text="ipData.isp"></span></div>
-                                                                                            <div><span class="text-xs text-base/60">Organization:</span> <span class="font-semibold text-sm" x-text="ipData.org"></span></div>
-                                                                                            <div><span class="text-xs text-base/60">AS:</span> <span class="font-semibold text-sm" x-text="ipData.as"></span></div>
-                                                                                            <div><span class="text-xs text-base/60">Coordinates:</span> <span class="font-semibold text-sm font-mono" x-text="ipData.lat + ', ' + ipData.lon"></span></div>
-                                                                                    </div>
-                                                                                </div>
-                                                                                    <div x-show="ipData.lat && ipData.lon" class="mt-4">
-                                                                                    <p class="text-xs text-base/60 mb-2">Location Map:</p>
-                                                                                        <div class="rounded-lg border border-neutral/20 overflow-hidden" style="height: 256px;">
-                                                                                            <iframe x-bind:src="'https://www.openstreetmap.org/export/embed.html?bbox=' + encodeURIComponent((parseFloat(ipData.lon) - 0.01) + ',' + (parseFloat(ipData.lat) - 0.01) + ',' + (parseFloat(ipData.lon) + 0.01) + ',' + (parseFloat(ipData.lat) + 0.01)) + '&layer=mapnik&marker=' + encodeURIComponent(ipData.lat + ',' + ipData.lon)" style="width: 100%; height: 100%; border: 0;" frameborder="0" loading="lazy" allowfullscreen title="Location Map"></iframe>
-                                                                                    </div>
-                                                                                </div>
-                                                                            </div>
-                                                                        </template>
-                                                                        <template x-if="ipData && (ipData.status === 'fail' || ipData.status === 'error')">
-                                                                            <div class="text-center py-4">
-                                                                                <p class="text-sm text-warning" x-text="ipData.message || 'Failed to load location data'"></p>
-                                                                            </div>
-                                                                        </template>
-                                                                    </div>
-                                                                </div>
-                                                            </div>
-                                                        </div>
-                                                    </div>
-                                                    @endif
-                                            </div>
                                         </td>
                                             <td class="py-3 px-4 text-sm">{{ strtoupper($rule['proto'] ?? ($rule['macro'] ?? 'ANY')) }}</td>
                                             <td class="py-3 px-4 text-sm font-mono text-xs">{{ $rule['dport'] ?? $rule['sport'] ?? '-' }}</td>