From 9f257b83e68aafb0745dbfc958e7bafa7a015a83 Mon Sep 17 00:00:00 2001
From: karim <karim@gabrielevarano.ch>
Date: Thu, 21 May 2026 00:51:13 +0200
Subject: [PATCH] Text-Bold/Italic-Toggle: face-Suffix-Stripping + FromQuartet
 zuerst
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

User: Bold/Kursiv liessen sich nicht zurueck auf normal stellen.

Diagnose: te.Font.QuartetName kann je nach RhinoCommon-Version den Bold/
Italic-Suffix im Namen mitfuehren (z.B. "Helvetica-Bold"). Dann liest
read_selection_settings face="Helvetica-Bold" → wird in updateTs ans
Backend zurueckgeschickt → _apply_font ruft FindOrCreate("Helvetica-Bold",
False, False) → das matcht intern wieder die Bold-Variante = bleibt fett.

Fix in _apply_font:
- Suffix-Stripping: -Bold, -Italic, -Oblique, -BoldItalic etc. werden vom
  face-String entfernt damit nur die Base-Family ("Helvetica") bleibt
- FromQuartetProperties zuerst (konstruiert Font direkt, unabhaengig vom
  FontTable-Cache). FindOrCreate als Fallback.
- Diagnostic print: "[TEXT] _apply_font face=... bold=... italic=..."
  damit sich nachvollziehen laesst was tatsaechlich angewendet wird

Plus textSettings/textStyles im State-Sig hinzugefuegt damit Idle-Pushes
Aenderungen nicht dedupelt verschlucken.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
---
 rhino/oberleiste.py  |  6 +++++-
 rhino/text_create.py | 46 +++++++++++++++++++++++++++++---------------
 2 files changed, 36 insertions(+), 16 deletions(-)

diff --git a/rhino/oberleiste.py b/rhino/oberleiste.py
index b50eea8..3c7a29a 100644
--- a/rhino/oberleiste.py
+++ b/rhino/oberleiste.py
@@ -1264,8 +1264,12 @@ class OberleisteBridge(panel_base.BaseBridge):
             _names_tuple, _active_comb,
             info.get("masseActiveId"),
             tuple((p.get("id"), p.get("name")) for p in (info.get("massePresets") or [])),
-            # textSelectionSettings: dict → tuple fuer sig-Vergleich
+            # textSettings + textSelectionSettings + textStyleActive im sig
+            # damit Aenderungen ueber Idle-Pushes nicht dedupelt werden
+            tuple(sorted((info.get("textSettings") or {}).items())) if info.get("textSettings") else None,
             tuple(sorted((info.get("textSelectionSettings") or {}).items())) if info.get("textSelectionSettings") else None,
+            info.get("textStyleActiveId"),
+            len(info.get("textStyles") or []),
             info.get("lastSetView"),
             prompt,
         )
diff --git a/rhino/text_create.py b/rhino/text_create.py
index 9dc649c..dbab208 100644
--- a/rhino/text_create.py
+++ b/rhino/text_create.py
@@ -198,28 +198,44 @@ def _prompt_for_text(default=""):
 
 
 def _apply_font(te, face, bold, italic):
-    """Setzt Font auf TextEntity. Mehrere Fallback-Pfade fuer
-    verschiedene RhinoCommon-Versionen."""
-    doc = Rhino.RhinoDoc.ActiveDoc
-    # Pfad 1: FindOrCreate im FontTable + zuweisen
+    """Setzt Font auf TextEntity. FromQuartetProperties zuerst (sauberer —
+    konstruiert Font-Objekt direkt mit (face, bold, italic), unabhaengig
+    von einem evtl. schon im FontTable existierenden Eintrag mit gleichem
+    Namen aber anderen Flags). Fallback FindOrCreate.
+
+    `face` wird normalisiert (Bold/Italic-Suffixe entfernen) damit
+    QuartetNames wie "Helvetica-Bold" nicht den Quartet-Lookup blockieren."""
+    face = str(face or "Helvetica").strip()
+    bold = bool(bold)
+    italic = bool(italic)
+    # Suffix-Stripping (haeufige Endungen die manche Fonts in der QuartetName
+    # haben — wir wollen die Base-Family)
+    for suffix in ("-BoldItalic", "-BoldOblique", "-Bold", "-Italic",
+                    "-Oblique", " Bold Italic", " Bold Oblique",
+                    " Bold", " Italic", " Oblique"):
+        if face.endswith(suffix):
+            face = face[:-len(suffix)].strip()
+            break
+    print("[TEXT] _apply_font face={!r} bold={} italic={}".format(face, bold, italic))
+    # Pfad 1: FromQuartetProperties (direkter, keine FontTable-Cache-Bugs)
     try:
-        font_idx = doc.Fonts.FindOrCreate(face, bool(bold), bool(italic))
+        font = Rhino.DocObjects.Font.FromQuartetProperties(face, bold, italic)
+        if font is not None:
+            te.Font = font
+            return True
+    except Exception as ex:
+        print("[TEXT] FromQuartet:", ex)
+    # Pfad 2: doc.Fonts.FindOrCreate
+    try:
+        doc = Rhino.RhinoDoc.ActiveDoc
+        font_idx = doc.Fonts.FindOrCreate(face, bold, italic)
         if font_idx >= 0:
             font = doc.Fonts[font_idx]
             if font is not None:
                 te.Font = font
                 return True
     except Exception as ex:
-        print("[TEXT] font path 1:", ex)
-    # Pfad 2: statische FromQuartetProperties
-    try:
-        font = Rhino.DocObjects.Font.FromQuartetProperties(
-            face, bool(bold), bool(italic))
-        if font is not None:
-            te.Font = font
-            return True
-    except Exception as ex:
-        print("[TEXT] font path 2:", ex)
+        print("[TEXT] FindOrCreate:", ex)
     return False