ui: don't use old settings when reusing sboxes

ISO keyboards have two physical keys that ANSI does not — POUND (#) at
row 3 col 12 between the right-of-quote position and Enter, and
ISO_BACKSLASH (<) at row 4 col 1 between LShift and Z. The firmware
reports them as zones 47 and 97 on G915 ISO models, but MAIN_ISO only
*subtracted* the ANSI backslash at row 2 col 13 without ever adding
those two cells back. They fell through to the unmapped pool and got
dropped by the EXTRAS_ALLOWLIST phantom-zone filter, so PerKey lighting
silently left them undrawable (issue #3239 — German G915).

Add both cells to MAIN_ISO with the UK QWERTY labels (# and \\) as the
default, and override them in the regional layouts: # / < on QWERTZ DE,
* / < on AZERTY FR. UK QWERTY inherits the defaults.

ANSI is unaffected — MAIN_ANSI still omits 47/97 so they keep getting
filtered as phantoms on ANSI keyboards like the G515.

lib/logitech_receiver/device.py

@@ -706,6 +706,14 @@ class Device:
                 long_message=long,
                 protocol=self.protocol,
             )
+        if logger.isEnabledFor(logging.WARN):
+            logger.warning(
+                "%s: request failure for device %s %s %s",
+                self,
+                self.handle,
+                self.receiver,
+                self.receiver._devices if self.receiver else None,
+            )
 
     def feature_request(self, feature, function=0x00, *params, no_reply=False):
         if self.protocol >= 2.0:
@@ -730,6 +738,8 @@ class Device:
                     if sub_idx is not None:
                         return self.centurion_bridge_request(sub_idx, function, *params, no_reply=no_reply)
             return hidpp20.feature_request(self, feature, function, *params, no_reply=no_reply)
+        if logger.isEnabledFor(logging.WARN):
+            logger.warning("%s: feature request failure for device with protocol %s", self, self.protocol)
 
     # Max sub-message bytes in the first bridge fragment (for 0x51):
     # 64 - 1 (report ID) - 1 (cpl_len) - 1 (flags) - 2 (bridge prefix) - 2 (bridge hdr) = 57;

lib/logitech_receiver/hidpp20.py

@@ -1756,6 +1756,10 @@ def feature_request(device, feature, function=0x00, *params, no_reply=False):
         if feature in device.features:
             feature_index = device.features[feature]
             return device.request((feature_index << 8) + (function & 0xFF), *params, no_reply=no_reply)
+    if logger.isEnabledFor(logging.WARN):
+        logger.warning(
+            "%s: feature request failure for device online %s and features %s", device, device.online, device.features
+        )
 
 
 class Hidpp20:

lib/logitech_receiver/listener.py

@@ -96,6 +96,8 @@ class _ThreadedHandle:
     def __str__(self):
         if self._local:
             return str(int(self))
+        else:
+            return "None"
 
     def __repr__(self):
         return f"<_ThreadedHandle({self.path})>"

lib/logitech_receiver/receiver.py

@@ -171,6 +171,8 @@ class Receiver:
         self.pairing = Pairing()
         self.initialize(product_info)
         hidpp10.set_configuration_pending_flags(self, 0xFF)
+        if logger.isEnabledFor(logging.INFO):
+            logger.info("%s: init receiver: handle %s, path %s, serial %s", self, self.handle, self.path, self.serial)
 
     def initialize(self, product_info: dict):
         # read the receiver information subregister, so we can find out max_devices
@@ -185,6 +187,8 @@ class Receiver:
             self.max_devices = product_info.get("max_devices", 1)
 
     def close(self):
+        if logger.isEnabledFor(logging.INFO):
+            logger.info("%s: closing - handle %s %s", self, type(self.handle), self.handle)
         handle, self.handle = self.handle, None
         for _n, d in self._devices.items():
             if d:
@@ -599,9 +603,10 @@ receiver_class_mapping = {
 
 def create_receiver(low_level: LowLevelInterface, device_info, setting_callback=None) -> Optional[Receiver]:
     """Opens a Logitech Receiver found attached to the machine, by Linux device path."""
-
     try:
         handle = low_level.open_path(device_info.path)
+        if logger.isEnabledFor(logging.INFO):
+            logger.info("create receiver %s %s", handle, device_info)
         if handle:
             usb_id = device_info.product_id
             if isinstance(usb_id, str):

lib/solaar/cli/config.py

@@ -231,7 +231,7 @@ def run(receivers, args, _find_receiver, find_device):
         argl = ["config", dev.serial or dev.unitId, setting.name]
         argl.extend([a for a in [args.value_key, args.extra_subkey, args.extra2] if a is not None])
         args = yaml.dump(argl)
-        application.run(args)
+        application.run([args])
     else:
         if dev.persister and setting.persist:
             dev.persister[setting.name] = setting._value

lib/solaar/ui/config_panel.py

@@ -1153,6 +1153,7 @@ def update(device, is_online=None):
         k = (device_id[0], device_id[1], s.name)
         if k in _items:
             sbox = _items[k]
+            sbox.setting = s  # use setting from current version of device
         else:
             sbox = _create_sbox(s, device)
             if sbox is None:

lib/solaar/ui/perkey/layouts/_keyboard_base.py

@@ -157,11 +157,17 @@ MAIN_ANSI: tuple[Cell, ...] = (
     Cell(zone_id=108, row=5, col=13, group="main", label="Ctrl"),
 )
 
-# --- Main alpha block, ISO. Same as ANSI minus the row 2 col 13 backslash;
+# --- Main alpha block, ISO. Drops the row 2 col 13 backslash (zone 46 is the
-#     on ISO that position is the top half of the L-shape Enter, addressed
+#     upper half of the L-shape Enter on ISO, addressed by zone 37) and adds
-#     by zone 37 (the main Enter cell at row 3 col 13). Zone 46 is silently
+#     the two ISO-only keys: POUND (zone 47) at row 3 col 12 between ' and
-#     unaddressable on ISO layouts — same limitation as OpenRGB's UI.
+#     Enter, and ISO_BACKSLASH (zone 97) at row 4 col 1 between Shift and Z.
-MAIN_ISO: tuple[Cell, ...] = tuple(c for c in MAIN_ANSI if not (c.row == 2 and c.col == 13))
+#     Regional layouts override the labels to match local keycaps (# / < on
+#     QWERTZ, # / \ on UK QWERTY, * / < on AZERTY).
+_ISO_EXTRA_KEYS: tuple[Cell, ...] = (
+    Cell(zone_id=47, row=3, col=12, group="main", label="#"),
+    Cell(zone_id=97, row=4, col=1, group="main", label="\\"),
+)
+MAIN_ISO: tuple[Cell, ...] = tuple(c for c in MAIN_ANSI if not (c.row == 2 and c.col == 13)) + _ISO_EXTRA_KEYS
 
 # --- Curated allowlist for unmapped device zones surfaced in the bottom strip.
 #     G-keys, logo, media, brightness — the canonical "extras" Logitech firmware

lib/solaar/ui/perkey/layouts/keyboard_iso_azerty.py

@@ -56,6 +56,8 @@ _OVERRIDES: dict[int, str] = {
     51: ";",  # ,-position → semicolon
     52: ":",  # .-position → colon
     53: "!",  # /-position → exclamation
+    47: "*",  # POUND key (row 3 col 12) — French * / µ
+    97: "<",  # ISO_BACKSLASH (row 4 col 1), between Shift and W
 }
 
 

lib/solaar/ui/perkey/layouts/keyboard_iso_qwertz.py

@@ -38,6 +38,8 @@ _OVERRIDES: dict[int, str] = {
     49: "Ä",  # row 3 col 11
     26: "Y",  # row 4 col 2 — Y/Z swap
     53: "-",  # row 4 col 11
+    47: "#",  # POUND key (row 3 col 12), between Ä and Enter
+    97: "<",  # ISO_BACKSLASH (row 4 col 1), between Shift and Y
 }