Window-Layout Auto-Apply: skip wenn bereits persistent angewendet
Cold-Start-Perf: das Auto-Apply von DOSSIERUIV0.2 lief bei jedem Rhino- Start einmal — Rhino's Window-Layout-Restore mountet dabei mehrere Panels neu (zweite Mount-Welle, ~70 ms gemessen + Rhino-internes Re- Layout). Rhino merkt sich die Window-Anordnung aber selbst persistent nach dem ersten erfolgreichen Apply. Loesung: zweite Persistenz-Ebene neben sticky. - sticky["_dossier_layout_applied"] bleibt fuer process-lifetime (verhindert Endlos-Loops bei Re-Mounts) - cfg["windowLayoutLastApplied"] in dossier_settings.json — bei naechstem Cold-Start wird verglichen: wenn name unveraendert → skip Apply-Cmd - _mark_layout_applied() helper schreibt nach erfolgreichem Apply - Alle 4 Apply-Pfade umgestellt: _on_ready (cold-start), open_settings_dialog (Eto-Button), APPLY_LAYOUT-Message (React), pendingApplyLayout (Launcher) Edge cases: - User aendert Layout-Name → last_applied != name → Re-Apply trotzdem - User klickt "Jetzt anwenden" → marker wird aktualisiert - Mac Rhino verliert Window-Anordnung → User muss manuell triggern (settings-button "Jetzt anwenden"); flag verhindert nicht das, nur das automatische Re-Apply Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
+37
-8
@@ -360,6 +360,22 @@ def _layout_name_to_guid(name):
|
|||||||
return None
|
return None
|
||||||
|
|
||||||
|
|
||||||
|
def _mark_layout_applied(name):
|
||||||
|
"""Schreibt den zuletzt erfolgreich angewendeten Layout-Namen in die
|
||||||
|
Launcher-settings.json. Spaetere Cold-Starts vergleichen und skippen
|
||||||
|
den Apply-Cmd wenn der Layout-Name unveraendert ist (Rhino merkt sich
|
||||||
|
die Window-Anordnung dann selbst persistent)."""
|
||||||
|
if not name: return
|
||||||
|
try:
|
||||||
|
cfg = _settings_load()
|
||||||
|
if cfg.get("windowLayoutLastApplied") == name:
|
||||||
|
return
|
||||||
|
cfg["windowLayoutLastApplied"] = name
|
||||||
|
_settings_save(cfg)
|
||||||
|
except Exception as ex:
|
||||||
|
print("[OBERLEISTE] _mark_layout_applied:", ex)
|
||||||
|
|
||||||
|
|
||||||
def _apply_window_layout(name):
|
def _apply_window_layout(name):
|
||||||
"""Wendet ein benanntes Window-Layout an. Probiert mehrere Wege weil
|
"""Wendet ein benanntes Window-Layout an. Probiert mehrere Wege weil
|
||||||
Mac Rhino 8 keine offizielle Python-API dafuer exponiert und die
|
Mac Rhino 8 keine offizielle Python-API dafuer exponiert und die
|
||||||
@@ -516,7 +532,8 @@ def open_settings_dialog():
|
|||||||
btn_apply.Text = "Jetzt anwenden"
|
btn_apply.Text = "Jetzt anwenden"
|
||||||
def _on_apply(s, e):
|
def _on_apply(s, e):
|
||||||
if state["defaultLayout"]:
|
if state["defaultLayout"]:
|
||||||
_apply_window_layout(state["defaultLayout"])
|
if _apply_window_layout(state["defaultLayout"]):
|
||||||
|
_mark_layout_applied(state["defaultLayout"])
|
||||||
btn_apply.Click += _on_apply
|
btn_apply.Click += _on_apply
|
||||||
|
|
||||||
btn_save = _ef.Button()
|
btn_save = _ef.Button()
|
||||||
@@ -814,17 +831,27 @@ class OberleisteBridge(panel_base.BaseBridge):
|
|||||||
self._dm_sent = False
|
self._dm_sent = False
|
||||||
self._commands_sent = False
|
self._commands_sent = False
|
||||||
# Default-Window-Layout anwenden, wenn aktiviert und noch nicht in
|
# Default-Window-Layout anwenden, wenn aktiviert und noch nicht in
|
||||||
# dieser Session geschehen (sticky-flag verhindert Endlos-Schleifen
|
# dieser Session geschehen. Doppelte Persistenz:
|
||||||
# falls die Layout-Restoration unsere Panels neu mountet). Layout-Name
|
# 1) `sticky["_dossier_layout_applied"]` — process-lifetime, schuetzt
|
||||||
# wird vom Launcher unter `windowLayout` geschrieben; Legacy-Key
|
# vor Endlos-Loops wenn die Layout-Restoration Panels re-mountet.
|
||||||
# `defaultLayout` wird in _settings_load() bereits normalisiert.
|
# 2) `cfg["windowLayoutLastApplied"]` — file-persistent, schuetzt vor
|
||||||
|
# re-apply bei jedem Cold-Start. Rhino merkt sich die Window-
|
||||||
|
# Anordnung selbststaendig nach dem ersten erfolgreichen Apply;
|
||||||
|
# spaetere Starts brauchen den Layout-Cmd nicht mehr.
|
||||||
try:
|
try:
|
||||||
cfg = _settings_load()
|
cfg = _settings_load()
|
||||||
if not sc.sticky.get("_dossier_layout_applied"):
|
if not sc.sticky.get("_dossier_layout_applied"):
|
||||||
layout_name = cfg.get("windowLayout") or cfg.get("defaultLayout")
|
layout_name = cfg.get("windowLayout") or cfg.get("defaultLayout")
|
||||||
if cfg.get("autoApplyLayout") and layout_name:
|
if cfg.get("autoApplyLayout") and layout_name:
|
||||||
sc.sticky["_dossier_layout_applied"] = True
|
sc.sticky["_dossier_layout_applied"] = True
|
||||||
_apply_window_layout(layout_name)
|
last_applied = cfg.get("windowLayoutLastApplied") or ""
|
||||||
|
if last_applied == layout_name:
|
||||||
|
print("[OBERLEISTE] Window-Layout '{}' bereits "
|
||||||
|
"persistent angewendet — skip Cold-Start-Apply "
|
||||||
|
"(spart Panel-Re-Mount-Welle)".format(layout_name))
|
||||||
|
else:
|
||||||
|
if _apply_window_layout(layout_name):
|
||||||
|
_mark_layout_applied(layout_name)
|
||||||
# Viewport-Colors einmalig pro Session auto-applien (wenn aktiviert)
|
# Viewport-Colors einmalig pro Session auto-applien (wenn aktiviert)
|
||||||
if (cfg.get("autoApplyViewColors") and
|
if (cfg.get("autoApplyViewColors") and
|
||||||
not sc.sticky.get("_dossier_view_colors_applied")):
|
not sc.sticky.get("_dossier_view_colors_applied")):
|
||||||
@@ -1033,7 +1060,8 @@ class OberleisteBridge(panel_base.BaseBridge):
|
|||||||
self._send_settings_state()
|
self._send_settings_state()
|
||||||
elif t == "APPLY_LAYOUT":
|
elif t == "APPLY_LAYOUT":
|
||||||
name = (p.get("name") or "").strip()
|
name = (p.get("name") or "").strip()
|
||||||
if name: _apply_window_layout(name)
|
if name and _apply_window_layout(name):
|
||||||
|
_mark_layout_applied(name)
|
||||||
elif t == "SAVE_LAYOUT_PREF":
|
elif t == "SAVE_LAYOUT_PREF":
|
||||||
cfg = _settings_load()
|
cfg = _settings_load()
|
||||||
if "windowLayout" in p:
|
if "windowLayout" in p:
|
||||||
@@ -1148,7 +1176,8 @@ class OberleisteBridge(panel_base.BaseBridge):
|
|||||||
pend_layout = cfg.get("pendingApplyLayout")
|
pend_layout = cfg.get("pendingApplyLayout")
|
||||||
if isinstance(pend_layout, str) and pend_layout:
|
if isinstance(pend_layout, str) and pend_layout:
|
||||||
print("[OBERLEISTE] pendingApplyLayout:", pend_layout)
|
print("[OBERLEISTE] pendingApplyLayout:", pend_layout)
|
||||||
_apply_window_layout(pend_layout)
|
if _apply_window_layout(pend_layout):
|
||||||
|
cfg["windowLayoutLastApplied"] = pend_layout
|
||||||
cfg.pop("pendingApplyLayout", None)
|
cfg.pop("pendingApplyLayout", None)
|
||||||
mutated = True
|
mutated = True
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user