fix: zjframes only working on first tab

fixes #109

Author: dj95

.gitignore

@@ -1,3 +1,4 @@
 /target
 .direnv/
 .zjstatus.log
+.zjframes.log

src/bin/zjframes.rs

@@ -32,7 +32,7 @@ fn init_tracing() {
     use std::fs::File;
     use tracing_subscriber::{layer::SubscriberExt, util::SubscriberInitExt};
 
-    let file = File::create("/host/.zjstatus.log");
+    let file = File::create("/host/.zjframes.log");
     let file = match file {
         Ok(file) => file,
         Err(error) => panic!("Error: {:?}", error),
@@ -143,6 +143,7 @@ impl State {
                     &self.state.panes,
                     &self.state.mode,
                     get_plugin_ids(),
+                    true,
                 );
             }
             Event::PaneUpdate(pane_info) => {
@@ -159,6 +160,7 @@ impl State {
                     &self.state.panes,
                     &self.state.mode,
                     get_plugin_ids(),
+                    true,
                 );
             }
             Event::PermissionRequestResult(result) => {
@@ -181,6 +183,7 @@ impl State {
                         &current_session.panes,
                         &self.state.mode,
                         get_plugin_ids(),
+                        true,
                     );
                 }
             }

src/bin/zjstatus.rs

@@ -208,6 +208,7 @@ impl State {
                     &pane_info,
                     &self.state.mode,
                     get_plugin_ids(),
+                    false,
                 );
 
                 self.state.panes = pane_info;
@@ -267,6 +268,7 @@ impl State {
                         &current_session.panes,
                         &self.state.mode,
                         get_plugin_ids(),
+                        false,
                     );
                 }
 

src/frames.rs

@@ -9,6 +9,7 @@ pub fn hide_frames_conditionally(
     pane_info: &PaneManifest,
     mode_info: &ModeInfo,
     plugin_pane_id: PluginIds,
+    is_zjframes: bool,
 ) {
     if !cfg_hide_frames_for_single_pane
         && !cfg_hide_frames_except_for_search
@@ -24,8 +25,11 @@ pub fn hide_frames_conditionally(
 
     // check if we are running for the current tab since one plugin will run for
     // each tab. If we do not prevent execution, the screen will start to flicker
-    // 'cause every plugin will try to toggle the frames
-    if !is_plugin_for_current_tab(&panes, plugin_pane_id) {
+    // 'cause every plugin will try to toggle the frames.
+    //
+    // This is only relevant for zjstatus, not zjframes; as zjframes only
+    // runs once per session.
+    if !is_plugin_for_current_tab(&panes, plugin_pane_id) && !is_zjframes {
         return;
     }
 

tests/zjframes/config.kdl

@@ -221,7 +221,7 @@ plugins {
 // Plugins to load in the background when a new session starts
 load_plugins {
     "file:./target/wasm32-wasip1/debug/zjframes.wasm" {
-        hide_frame_for_single_pane       "false"
+        hide_frame_for_single_pane       "true"
         hide_frame_except_for_search     "true"
         hide_frame_except_for_fullscreen "true"
     }
@@ -259,7 +259,15 @@ load_plugins {
 //   - true (default)
 //   - false
 //
-// pane_frames true
+pane_frames true
+
+
+ui {
+  pane_frames {
+    hide_session_name true
+    rounded_corners true
+  }
+}
 
 // Toggle between having Zellij lay out panes according to a predefined set of layouts whenever possible
 // Options:
@@ -274,7 +282,7 @@ load_plugins {
 //   - true (default)
 //   - false
 //
-// session_serialization false
+session_serialization false
 
 // Whether pane viewports are serialized along with the session, default is false
 // (Requires restart)