Merge pull request #42 from SMLunchen/fix_vector_show_node_on_map

- bugfix node name column not resizable

Author: SMLunchen

CHANGELOG.md

@@ -7,6 +7,36 @@ und dieses Projekt folgt [Semantic Versioning](https://semver.org/lang/de/).
 
 ---
 
+## [1.6.2.2] - 2026-08-14
+
+### ✨ Hinzugefügt / 🔧 Geändert
+
+#### 🗄️ NodeDB-Reset: Abfrage-Dialog mit Optionen
+- Der Button „NodeDB zurücksetzen" (Geräte-Einstellungen **und** Remote-Admin) öffnet jetzt einen Dialog mit zwei Häkchen statt einer simplen Ja/Nein-Abfrage:
+  - **Favoriten ebenfalls löschen** — steuert das `nodedb_reset`-Bool: Standard behält Favoriten (`true`), angehakt löscht sie (`false`). Firmware-Abgleich ergab: der Bool-Wert ist genau `keepFavorites` in `resetNodes()`. Hinweis im Dialog: Router/Client-Base behalten Favoriten firmwareseitig immer.
+  - **Interne Node-DB dieses Clients ebenfalls zurücksetzen** — leert zusätzlich die lokale Node-Liste (In-Memory) + Karten-Pins; Nachrichten-/Telemetrie-History bleibt.
+- **Korrektur der Beschreibungen:** Texte sagten „alle bekannten Nodes werden vergessen", obwohl Favoriten erhalten blieben — jetzt korrekt formuliert. Der Geräte-Reset löst wie zuvor einen **Reboot** aus (kein Shutdown).
+- Der hartcodierte Dialogtext wurde durch i18n-Strings (DE/EN) ersetzt.
+
+### 🐛 Behoben
+
+#### 🗺️ „Auf Karte zeigen" zentrierte die Vector-Map nicht
+- Bei aktiver **Vector-Map** (MapLibre/WebView2) öffnete „Auf Karte zeigen" zwar die Karte, sprang aber **nicht** auf die Node-Position — die Zentrierung lief nur über die Mapsui-**Rastermap**. Jetzt zentriert ein gemeinsamer Helfer (`CenterMapOnNode`) **beide** Karten (Raster via `CenterOnAndZoomTo`, Vector via `setCenter`). Betrifft alle drei Einstiege (Node-Kontextmenü, Nachrichten-Kontextmenü, Alert-Button). Öffnet der Klick die Vector-Map erstmalig, wird das Ziel gepuffert und nach dem Laden angewandt.
+
+#### 🔓 PKI-DMs werden sofort entschlüsselt, wenn wir den Key haben (Flag-unabhängig)
+- Der sofortige client-seitige PKI-Entschlüsselungsversuch war an `packet.PkiEncrypted` gekoppelt. Die Firmware setzt dieses Flag aber auf `false`, sobald der **eigene** Node nicht entschlüsseln konnte (Router.cpp:814) — z.B. nach einem **NodeDB-Reset**, wenn dessen nodedb den Sender-Key nicht mehr hat. Dadurch wurden PKI-DMs gepuffert und nicht entschlüsselt, **obwohl der Client den Key besitzt**.
+- Jetzt versucht der Client PKI-Entschlüsselung für **jede DM an uns**, sobald Private- und Sender-Key vorliegen — unabhängig vom Flag (sicher, da AES-CCM tag-verifiziert ist). Telemetrie/DMs von Nodes, deren Key wir kennen (der Node aber nach Reset nicht), gehen so sofort auf statt im Puffer zu landen.
+
+#### 📨 Request-Antworten werden nach Portnum verarbeitet (nicht mehr als „verschlüsselte DM")
+- Antworten auf `want_response`-Anfragen (UserInfo/Position/Telemetrie/PaxCounter) kommen teils verschlüsselt zurück: **Telemetrie/Pax** PKI-verschlüsselt (der Node hat unseren Key aus der Anfrage, uns fehlt seiner → Henne-Ei), **NodeInfo/Position** kanal-verschlüsselt (Firmware schließt diese Portnums von PKI aus, Router.cpp:1053). Konnte unser Node sie nicht entschlüsseln, landeten sie pauschal als „[Encrypted message]" im Chat und wurden beim Entschlüsseln nur akzeptiert, wenn es Text war.
+- Jetzt: ein gepuffertes Paket wird nach Entschlüsselung **nach Portnum geroutet** (`DispatchLateDecrypted` → `RouteDecodedData`) — NodeInfo/Position/Telemetrie/… gehen in die normale Verarbeitung (Node/Key/Telemetrie werden übernommen), nur echter Text (Portnum 1) erscheint als DM.
+- **Zusätzlich:** Kommt eine verschlüsselte Antwort kurz nach einer **expliziten** Info-Anfrage an denselben Node (≤30 s), wird sie als Request-Antwort erkannt, gepuffert und **nicht** mehr als Chat-Platzhalter angezeigt (und löst keine weitere NodeInfo-Anfrage aus). Hinweis: `pki=False` im Log ist irreführend — die Firmware setzt das Flag zurück, sobald der eigene Node nicht entschlüsseln konnte (Router.cpp:814).
+
+#### 📏 Node-Namen-Spalte folgt jetzt der Spaltenbreite
+- Die Namensspalte in der Node-Tabelle war hart auf 155 px gedeckelt (`MaxWidth`) und saß in einem `StackPanel` — beim Verbreitern der Spalte wurde der Name trotzdem abgeschnitten. Das Cell-Template nutzt jetzt ein `Grid` (Auto + `*`) ohne `MaxWidth`, und die Zeilen strecken sich auf die Spaltenbreite (`HorizontalContentAlignment=Stretch`). Der Name folgt jetzt der Spaltenbreite (Ellipsis) und zeigt per Tooltip den vollen Namen.
+
+---
+
 ## [1.6.2.1] - 2026-08-13
 
 ### ✨ Hinzugefügt

MeshhessenClient/MainWindow.Map.cs

@@ -298,6 +298,13 @@ await VectorMapView.CoreWebView2.ExecuteScriptAsync(
             // Restore cached line overlays (traceroutes, paths) e.g. after raster->vector switch
             foreach (var (key, json) in _vectorLineJson.ToList())
                 ExecVectorScript($"setLines({JsonSerializer.Serialize(key)}, {json})");
+
+            // A "Show on map" click that opened the map applies its center now that JS is callable.
+            if (_pendingVectorCenter is { } pc)
+            {
+                _pendingVectorCenter = null;
+                ExecVectorScript($"setCenter({pc.Lon.ToString(ci)}, {pc.Lat.ToString(ci)}, {pc.Zoom})");
+            }
         }
         catch (Exception ex)
         {
@@ -589,6 +596,49 @@ private async void ExecVectorScript(string script)
         catch (Exception ex) { Services.Logger.WriteLine($"[VectorMap] script error: {ex.Message}"); }
     }
 
+    /// <summary>
+    /// Centers whichever map is currently active on a node position. Handles both the
+    /// Mapsui raster map (<see cref="_map"/>) and the MapLibre vector map (WebView2) —
+    /// the raster-only path used to leave the vector map un-centered on "Show on map".
+    /// </summary>
+    private void CenterMapOnNode(double lat, double lon)
+    {
+        var nodePos = SphericalMercator.FromLonLat(lon, lat);
+        if (_map != null)
+        {
+            _map.Navigator.CenterOnAndZoomTo(new MPoint(nodePos.x, nodePos.y), 76.0);
+            MapControl.Refresh();
+        }
+        if (UseVectorMap)
+        {
+            const int zoom = 12;
+            if (_vectorMapReady)
+            {
+                var ci = System.Globalization.CultureInfo.InvariantCulture;
+                ExecVectorScript($"setCenter({lon.ToString(ci)}, {lat.ToString(ci)}, {zoom})");
+            }
+            else
+            {
+                // Map tab opened for the first time by this very click — apply once it's ready.
+                _pendingVectorCenter = (lon, lat, zoom);
+            }
+        }
+    }
+
+    /// <summary>Removes every node pin from both maps (raster features + vector). Assumes
+    /// <c>_allNodes</c> is already cleared so the vector push writes an empty set.</summary>
+    private void ClearAllNodePinsFromMap()
+    {
+        _nodeFeatures.Clear();
+        if (_nodeLayer != null)
+        {
+            _nodeLayer.Features = _nodeFeatures;
+            _nodeLayer.DataHasChanged();
+        }
+        MapControl?.Refresh();
+        PushNodePinsToVectorMap();
+    }
+
     private static string CssColor(Mapsui.Styles.Color c) =>
         $"rgba({c.R},{c.G},{c.B},{(c.A / 255.0).ToString("0.##", System.Globalization.CultureInfo.InvariantCulture)})";
 

MeshhessenClient/MainWindow.NodeList.cs

@@ -64,12 +64,7 @@ private void NodeContextMenu_ShowOnMap_Click(object sender, RoutedEventArgs e)
         }
 
         MainTabs.SelectedIndex = 3;
-        var nodePos = SphericalMercator.FromLonLat(node.Longitude.Value, node.Latitude.Value);
-        if (_map != null)
-        {
-            _map.Navigator.CenterOnAndZoomTo(new MPoint(nodePos.x, nodePos.y), 76.0);
-            MapControl.Refresh();
-        }
+        CenterMapOnNode(node.Latitude.Value, node.Longitude.Value);
     }
 
     private void NodeFilterTextBox_TextChanged(object sender, System.Windows.Controls.TextChangedEventArgs e)

MeshhessenClient/MainWindow.xaml

@@ -1219,6 +1219,8 @@
                                 <Setter Property="MinHeight" Value="42"/>
                                 <Setter Property="BorderThickness" Value="0"/>
                                 <Setter Property="Padding" Value="0,2"/>
+                                <!-- Cells fill their column width so name trims to the column and follows resize -->
+                                <Setter Property="HorizontalContentAlignment" Value="Stretch"/>
                                 <Style.Triggers>
                                     <!-- Ungerade Zeilen: dezentes Grau -->
                                     <Trigger Property="ItemsControl.AlternationIndex" Value="1">
@@ -1276,8 +1278,14 @@
                                     </GridViewColumn.Header>
                                     <GridViewColumn.CellTemplate>
                                         <DataTemplate>
-                                            <StackPanel Orientation="Horizontal" VerticalAlignment="Center">
-                                                <Border Width="28" Height="28" CornerRadius="14"
+                                            <!-- Grid (Auto + *) so the name tracks the column width and trims to it;
+                                                 a StackPanel measures children with infinite width and ignores resize. -->
+                                            <Grid VerticalAlignment="Center" ClipToBounds="True">
+                                                <Grid.ColumnDefinitions>
+                                                    <ColumnDefinition Width="Auto"/>
+                                                    <ColumnDefinition Width="*"/>
+                                                </Grid.ColumnDefinitions>
+                                                <Border Grid.Column="0" Width="28" Height="28" CornerRadius="14"
                                                         Background="{Binding AvatarColor, Converter={StaticResource HexToColorConverter}}"
                                                         Margin="0,0,7,0">
                                                     <TextBlock Text="{Binding AvatarLabel}"
@@ -1286,12 +1294,13 @@
                                                                FontSize="9" FontWeight="Bold"
                                                                Foreground="White"/>
                                                 </Border>
-                                                <emoji:TextBlock Text="{Binding Name}"
+                                                <emoji:TextBlock Grid.Column="1" Text="{Binding Name}"
                                                            VerticalAlignment="Center"
+                                                           HorizontalAlignment="Left"
                                                            FontWeight="SemiBold"
                                                            TextTrimming="CharacterEllipsis"
-                                                           MaxWidth="155"/>
-                                            </StackPanel>
+                                                           ToolTip="{Binding Name}"/>
+                                            </Grid>
                                         </DataTemplate>
                                     </GridViewColumn.CellTemplate>
                                 </GridViewColumn>

MeshhessenClient/MainWindow.xaml.cs

@@ -84,6 +84,7 @@ public partial class MainWindow : Window
     private Services.VectorTileCacheService? _vectorTileCache;
     private bool _vectorMapInitStarted = false;  // CoreWebView2 init/navigation kicked off
     private bool _vectorMapReady = false;        // map.html loaded, JS API callable
+    private (double Lon, double Lat, int Zoom)? _pendingVectorCenter;  // "Show on map" fired before the vector map was ready
     private MemoryLayer? _nodeLayer;
     private MemoryLayer? _myPosLayer;
     private readonly List<IFeature> _nodeFeatures = new();
@@ -193,6 +194,7 @@ public MainWindow()
         _protocolService.MessageReceived += OnMessageReceived;
         _protocolService.PkiMessageDecrypted += OnPkiMessageDecrypted;
         _protocolService.NodeInfoReceived += OnNodeInfoReceived;
+        _protocolService.NodeDbCleared += OnNodeDbCleared;
         _protocolService.ChannelInfoReceived += OnChannelInfoReceived;
         _protocolService.LoRaConfigReceived += OnLoRaConfigReceived;
         _protocolService.DeviceInfoReceived += OnDeviceInfoReceived;
@@ -897,6 +899,7 @@ await Task.Run(() =>
                 _protocolService.MessageReceived += OnMessageReceived;
                 _protocolService.PkiMessageDecrypted += OnPkiMessageDecrypted;
                 _protocolService.NodeInfoReceived += OnNodeInfoReceived;
+                _protocolService.NodeDbCleared += OnNodeDbCleared;
                 _protocolService.ChannelInfoReceived += OnChannelInfoReceived;
                 _protocolService.LoRaConfigReceived += OnLoRaConfigReceived;
                 _protocolService.DeviceInfoReceived += OnDeviceInfoReceived;
@@ -1609,6 +1612,7 @@ private async Task TryReconnectAsync()
                 _protocolService.MessageReceived += OnMessageReceived;
                 _protocolService.PkiMessageDecrypted += OnPkiMessageDecrypted;
                 _protocolService.NodeInfoReceived += OnNodeInfoReceived;
+                _protocolService.NodeDbCleared += OnNodeDbCleared;
                 _protocolService.ChannelInfoReceived += OnChannelInfoReceived;
                 _protocolService.LoRaConfigReceived += OnLoRaConfigReceived;
                 _protocolService.DeviceInfoReceived += OnDeviceInfoReceived;
@@ -1964,6 +1968,26 @@ private void OnNodeInfoReceived(object? sender, Models.NodeInfo node)
         });
     }
 
+    /// <summary>Internal node DB was cleared (user-triggered) — drop the UI node list and map pins.</summary>
+    private void OnNodeDbCleared(object? sender, EventArgs e)
+    {
+        Dispatcher.BeginInvoke(() =>
+        {
+            try
+            {
+                _nodes.Clear();
+                _allNodes.Clear();
+                ApplyNodeSortAndFilterCore();
+                ClearAllNodePinsFromMap();
+                Services.Logger.WriteLine("[Nodes] Internal node DB cleared — UI list and map pins reset");
+            }
+            catch (Exception ex)
+            {
+                Services.Logger.WriteLine($"Error clearing node UI: {ex.Message}");
+            }
+        });
+    }
+
     private void OnChannelInfoReceived(object? sender, ChannelInfo channel)
     {
         // Async Update - blockiert nicht den UI Thread
@@ -2605,12 +2629,7 @@ private void MessageContextMenu_ShowOnMap_Click(object sender, RoutedEventArgs e
         }
 
         MainTabs.SelectedIndex = 3;
-        var nodePos = SphericalMercator.FromLonLat(node.Longitude.Value, node.Latitude.Value);
-        if (_map != null)
-        {
-            _map.Navigator.CenterOnAndZoomTo(new MPoint(nodePos.x, nodePos.y), 76.0);
-            MapControl.Refresh();
-        }
+        CenterMapOnNode(node.Latitude.Value, node.Longitude.Value);
     }
 
     private void MessageContextMenu_SetColor_Click(object sender, RoutedEventArgs e)
@@ -3218,14 +3237,8 @@ private void ShowOnMap_Click(object sender, RoutedEventArgs e)
             // Switch to Map tab
             MainTabs.SelectedIndex = 3; // Map is tab index 3 (0=Messages, 1=Nodes, 2=Channels, 3=Map, 4=Settings, ...)
 
-            // Center map on node position with closer zoom
-            var nodePos = SphericalMercator.FromLonLat(node.Longitude.Value, node.Latitude.Value);
-            if (_map != null)
-            {
-                // Zoom level 12 (resolution ~76)
-                _map.Navigator.CenterOnAndZoomTo(new MPoint(nodePos.x, nodePos.y), 76.0);
-                MapControl.Refresh();
-            }
+            // Center whichever map is active (raster + vector) on the node position
+            CenterMapOnNode(node.Latitude.Value, node.Longitude.Value);
 
             // Close notification
             AlertNotificationBar.Visibility = Visibility.Collapsed;

MeshhessenClient/MeshhessenClient.csproj

@@ -18,9 +18,9 @@
     <PublishDir>..\public\</PublishDir>
 
     <AssemblyName>MeshhessenClient</AssemblyName>
-    <Version>1.6.2.1</Version>
-    <AssemblyVersion>1.6.2.1</AssemblyVersion>
-    <FileVersion>1.6.2.1</FileVersion>
+    <Version>1.6.2.2</Version>
+    <AssemblyVersion>1.6.2.2</AssemblyVersion>
+    <FileVersion>1.6.2.2</FileVersion>
     <Company>Meshtastic Community</Company>
     <Product>Meshhessen Client</Product>
     <Description>Offline-fähiger Windows Client für Meshtastic Geräte</Description>

MeshhessenClient/NodeConfigWindow.xaml.cs

@@ -918,18 +918,21 @@ private static string WindowsToPosixTz(TimeZoneInfo tz)
 
     private async void NodeDbReset_Click(object sender, RoutedEventArgs e)
     {
-        var result = MessageBox.Show(
-            "NodeDB zurücksetzen?\n\nDas löscht alle bekannten Nodes auf dem Gerät.\nDie eigene Konfiguration bleibt erhalten.\n\nDas Gerät wird danach neu starten.",
-            "NodeDB zurücksetzen",
-            MessageBoxButton.YesNo,
-            MessageBoxImage.Warning);
-        if (result != MessageBoxResult.Yes) return;
+        var dlg = new NodeDbResetDialog { Owner = this };
+        if (dlg.ShowDialog() != true) return;
 
         try
         {
-            await _protocolService.ResetNodeDbAsync();
-            UpdateStatus("NodeDB-Reset gesendet. Gerät startet neu...");
-            Logger.WriteLine("NodeDB reset sent");
+            // nodedb_reset value == keepFavorites (firmware: resetNodes(keepFavorites)).
+            // The checkbox asks to WIPE favorites, so keepFavorites = !WipeFavorites.
+            await _protocolService.ResetNodeDbAsync(keepFavorites: !dlg.WipeFavorites);
+
+            // Optionally also drop our own in-memory node cache.
+            if (dlg.ResetInternalDb)
+                _protocolService.ClearKnownNodes();
+
+            UpdateStatus(Loc("StrNcNodeDbResetSent"));
+            Logger.WriteLine($"NodeDB reset sent (wipeFavorites={dlg.WipeFavorites}, resetInternal={dlg.ResetInternalDb})");
         }
         catch (Exception ex)
         {

MeshhessenClient/NodeDbResetDialog.xaml

@@ -0,0 +1,39 @@
+<Window x:Class="MeshhessenClient.NodeDbResetDialog"
+        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
+        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
+        xmlns:ui="http://schemas.modernwpf.com/2019"
+        ui:WindowHelper.UseModernWindowStyle="True"
+        Title="{DynamicResource StrNodeDbResetTitle}"
+        Width="500"
+        SizeToContent="Height"
+        MinWidth="440"
+        ResizeMode="NoResize"
+        WindowStartupLocation="CenterOwner">
+    <StackPanel Margin="22">
+        <TextBlock Text="{DynamicResource StrNodeDbResetHeader}"
+                   FontWeight="SemiBold" FontSize="15" TextWrapping="Wrap" Margin="0,0,0,10"/>
+        <TextBlock Text="{DynamicResource StrNodeDbResetBody}"
+                   TextWrapping="Wrap" Margin="0,0,0,18"/>
+
+        <!-- Option 1: wipe favorites too -->
+        <CheckBox x:Name="WipeFavoritesCheck"
+                  Content="{DynamicResource StrNodeDbResetWipeFavorites}" Margin="0,0,0,4"/>
+        <TextBlock Text="{DynamicResource StrNodeDbResetFavoritesHint}"
+                   FontSize="11" Opacity="0.75"
+                   TextWrapping="Wrap" Margin="28,0,0,14"/>
+
+        <!-- Option 2: also reset our internal node DB -->
+        <CheckBox x:Name="ResetInternalCheck"
+                  Content="{DynamicResource StrNodeDbResetInternal}" Margin="0,0,0,4"/>
+        <TextBlock x:Name="InternalHint" Text="{DynamicResource StrNodeDbResetInternalHint}"
+                   FontSize="11" Opacity="0.75"
+                   TextWrapping="Wrap" Margin="28,0,0,18"/>
+
+        <StackPanel Orientation="Horizontal" HorizontalAlignment="Right">
+            <Button x:Name="CancelButton" Content="{DynamicResource StrNodeDbResetCancelBtn}"
+                    Width="110" Margin="0,0,8,0" Click="Cancel_Click" IsCancel="True"/>
+            <Button x:Name="OkButton" Content="{DynamicResource StrNodeDbResetConfirmBtn}"
+                    Width="150" Click="Ok_Click" IsDefault="True"/>
+        </StackPanel>
+    </StackPanel>
+</Window>