From f8c107b0874c0242bd13941590ab3ecb87a97fc9 Mon Sep 17 00:00:00 2001
From: Eduard Gert <kontakt@eduardgert.de>
Date: Tue, 26 May 2026 09:35:03 +0200
Subject: [PATCH] update peers filter, fix duplicate url open in dev

---
 .../modules/authentication/WaitingForBrowserDialog.tsx | 10 +++++++---
 client/ui/frontend/src/modules/peers/PeerFilters.tsx   |  4 +---
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/client/ui/frontend/src/modules/authentication/WaitingForBrowserDialog.tsx b/client/ui/frontend/src/modules/authentication/WaitingForBrowserDialog.tsx
index a702c9620..172f9fd47 100644
--- a/client/ui/frontend/src/modules/authentication/WaitingForBrowserDialog.tsx
+++ b/client/ui/frontend/src/modules/authentication/WaitingForBrowserDialog.tsx
@@ -1,4 +1,4 @@
-import { useCallback, useEffect } from "react";
+import { useCallback, useEffect, useRef } from "react";
 import { useTranslation } from "react-i18next";
 import { useSearchParams } from "react-router-dom";
 import { Dialogs, Events } from "@wailsio/runtime";
@@ -21,6 +21,7 @@ export default function WaitingForBrowserDialog() {
     const [params] = useSearchParams();
     const uri = params.get("uri") ?? "";
     const contentRef = useAutoSizeWindow<HTMLDivElement>(WINDOW_WIDTH);
+    const openedRef = useRef(false);
 
     const reportOpenFailure = useCallback(
         (e: unknown) => {
@@ -35,9 +36,12 @@ export default function WaitingForBrowserDialog() {
     // Open the system browser only after the dialog has mounted (which
     // means useAutoSizeWindow has called Window.Show). startLogin used to
     // fire OpenURL itself but the browser typically beat React's mount
-    // and landed on top of the still-hidden NetBird popup.
+    // and landed on top of the still-hidden NetBird popup. The ref guard
+    // keeps StrictMode's intentional double-invoke in dev (and any future
+    // remount) from launching two browser tabs.
     useEffect(() => {
-        if (!uri) return;
+        if (!uri || openedRef.current) return;
+        openedRef.current = true;
         Connection.OpenURL(uri).catch(reportOpenFailure);
     }, [uri, reportOpenFailure]);
 
diff --git a/client/ui/frontend/src/modules/peers/PeerFilters.tsx b/client/ui/frontend/src/modules/peers/PeerFilters.tsx
index 8710551cf..6564c5a37 100644
--- a/client/ui/frontend/src/modules/peers/PeerFilters.tsx
+++ b/client/ui/frontend/src/modules/peers/PeerFilters.tsx
@@ -28,9 +28,7 @@ export const PeerFilters = ({ value, onChange, counts }: Props) => {
             {filters.map((f) => (
                 <SwitchItem key={f.value} value={f.value} className={"flex-1"}>
                     {f.label}
-                    <span className={"font-normal text-nb-gray-200"}>
-                        {counts[f.value]}
-                    </span>
+                    <span className={"tabular-nums"}>({counts[f.value]})</span>
                 </SwitchItem>
             ))}
         </SwitchItemGroup>