This is an automated email from the git hooks/post-receive script.

git pushed a commit to branch tymux
in repository terminology.

View the commit online.

commit d5a0812d042007a6ef71d6856fdc9182c832934c
Author: [email protected] <[email protected]>
AuthorDate: Thu Mar 19 13:19:02 2026 -0600

    fix: address tymux indicator code review issues
    
    - Add _tymux_indicator_update to _term_bg_config so the indicator
      survives hot theme reloads (registered as theme_reload_func)
    - Add missing tymux,off program to sel/item theme group for symmetry
    - Fix stale comment: "2px" → "1px" in background.edc
    - Replace non-standard max: N -1 with max: N 99999 matching the
      project convention for unconstrained axis dimensions
    
    Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
---
 data/themes/default/background.edc | 6 +++---
 data/themes/default/sel.edc        | 8 ++++++--
 src/bin/win.c                      | 3 +++
 3 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/data/themes/default/background.edc b/data/themes/default/background.edc
index fdfa8d82..1dade79e 100644
--- a/data/themes/default/background.edc
+++ b/data/themes/default/background.edc
@@ -1588,13 +1588,13 @@ group { name: "terminology/background";
       // }}}
 
       ////////////////////////////////////////////////////////////////////
-      // tymux session indicator — 2px green bar on left edge (topmost)
+      // tymux session indicator — 1px green bar on left edge (topmost)
       part { name: "tymux_indicator"; type: RECT; mouse_events: 0;
          description { state: "default" 0.0;
             rel1 { relative: 0.0 0.0; }
             rel2 { relative: 0.0 1.0; }
             min: 1 0;
-            max: 1 -1;
+            max: 1 99999;
             fixed: 1 0;
             align: 0.0 0.5;
             color: 0 0 0 0;
@@ -1604,7 +1604,7 @@ group { name: "terminology/background";
             rel1 { relative: 0.0 0.0; }
             rel2 { relative: 0.0 1.0; }
             min: 1 0;
-            max: 1 -1;
+            max: 1 99999;
             fixed: 1 0;
             align: 0.0 0.5;
             color_class: ":tymux-indicator";
diff --git a/data/themes/default/sel.edc b/data/themes/default/sel.edc
index c2781f00..e42d0d88 100644
--- a/data/themes/default/sel.edc
+++ b/data/themes/default/sel.edc
@@ -264,7 +264,7 @@ group { name: "terminology/sel/item";
             rel1 { relative: 0.0 0.0; to: "terminology.content"; }
             rel2 { relative: 0.0 1.0; to: "terminology.content"; }
             min: 2 0;
-            max: 2 -1;
+            max: 2 99999;
             fixed: 1 0;
             align: 0.0 0.5;
             color: 0 0 0 0;
@@ -274,7 +274,7 @@ group { name: "terminology/sel/item";
             rel1 { relative: 0.0 0.0; to: "terminology.content"; }
             rel2 { relative: 0.0 1.0; to: "terminology.content"; }
             min: 2 0;
-            max: 2 -1;
+            max: 2 99999;
             fixed: 1 0;
             align: 0.0 0.5;
             color_class: ":tymux-indicator";
@@ -286,6 +286,10 @@ group { name: "terminology/sel/item";
          action: STATE_SET "on" 0.0;
          target: "tymux_indicator";
       }
+      program { signal: "tymux,off"; source: "terminology";
+         action: STATE_SET "default" 0.0;
+         target: "tymux_indicator";
+      }
 
       program { signal: "bell"; source: "terminology";
          action: STATE_SET "visible" 0.0;
diff --git a/src/bin/win.c b/src/bin/win.c
index af731e08..78c10b65 100644
--- a/src/bin/win.c
+++ b/src/bin/win.c
@@ -7272,6 +7272,9 @@ _term_bg_config(Term *term)
      }
    if (term->miniview_shown)
         elm_layout_signal_emit(term->bg, "miniview,on", "terminology");
+#ifdef HAVE_TYMUX
+   _tymux_indicator_update(term);
+#endif
 
    _term_media_update(term, term->config);
 }

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.

Reply via email to