Fix colored pane frames in mirrored sessions (#2625)

* server/panes/tiled: Fix colored frames

in mirrored sessions. Colored frames were previously ignored because
they were treated like floating panes when rendering tiled panes.

* CHANGELOG: Add PR #2625

* server/tab/unit: Fix unit tests for server.

6 files changed, 6 insertions, 9 deletions

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 71d30f44e..fe3ea7e13 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -8,6 +8,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 * fix(tab-bar,compact-bar): tab switching with mouse sometimes not working (https://github.com/zellij-org/zellij/pull/2587)
 * feat(status-bar): supermode to prevent colliding keybindings (https://github.com/zellij-org/zellij/pull/2619)
 * fix(rendering): occasional glitches while resizing (https://github.com/zellij-org/zellij/pull/2621)
+* fix(rendering): colored paneframes in mirrored sessions (https://github.com/zellij-org/zellij/pull/2625)
 
 ## [0.37.2] - 2023-06-20
 * hotfix: include theme files into binary (https://github.com/zellij-org/zellij/pull/2566)
diff --git a/zellij-server/src/panes/tiled_panes/mod.rs b/zellij-server/src/panes/tiled_panes/mod.rs
index 424053405..f0a6e8436 100644
--- a/zellij-server/src/panes/tiled_panes/mod.rs
+++ b/zellij-server/src/panes/tiled_panes/mod.rs
@@ -623,7 +623,7 @@ impl TiledPanes {
             { self.connected_clients.borrow().iter().copied().collect() };
         let multiple_users_exist_in_session = { self.connected_clients_in_app.borrow().len() > 1 };
         let mut client_id_to_boundaries: HashMap<ClientId, Boundaries> = HashMap::new();
-        let active_panes = if self.session_is_mirrored || floating_panes_are_visible {
+        let active_panes = if floating_panes_are_visible {
             HashMap::new()
         } else {
             self.active_panes
diff --git a/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__base_layout_is_included_in_swap_layouts.snap b/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__base_layout_is_included_in_swap_layouts.snap
index 5fbf2847a..97692e660 100644
--- a/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__base_layout_is_included_in_swap_layouts.snap
+++ b/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__base_layout_is_included_in_swap_layouts.snap
@@ -1,6 +1,5 @@
 ---
 source: zellij-server/src/tab/./unit/tab_integration_tests.rs
-assertion_line: 4762
 expression: snapshot
 ---
 00 (C): I am a tab bar                                                                                                           
@@ -20,7 +19,7 @@ expression: snapshot
 14 (C): │                                       ││                                      ││                                      │
 15 (C): │                                       ││                                      ││                                      │
 16 (C): │                                       ││                                      ││                                      │
-17 (C): └───────────────────────────────────────┘└──────────────────────────────────────┘└──────────────────────────────────────┘
+17 (C): └───────────────────────────────────────┘└──────────────────────────────────────┘└─ <ENTER> to run, <Ctrl-c> to exit ───┘
 18 (C): I am a                                                                                                                   
 19 (C): status bar                                                                                                               
 
diff --git a/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__layout_with_plugins_and_commands_swaped_properly.snap b/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__layout_with_plugins_and_commands_swaped_properly.snap
index c02cd98ca..4c31fe251 100644
--- a/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__layout_with_plugins_and_commands_swaped_properly.snap
+++ b/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__layout_with_plugins_and_commands_swaped_properly.snap
@@ -1,6 +1,5 @@
 ---
 source: zellij-server/src/tab/./unit/tab_integration_tests.rs
-assertion_line: 4992
 expression: snapshot
 ---
 00 (C): I am a                                                                                                                   
@@ -10,7 +9,7 @@ expression: snapshot
 04 (C): │                                              Waiting to run: command1                                                 │
 05 (C): │                                                                                                                       │
 06 (C): │                                          <ENTER> to run, <Ctrl-c> to exit                                             │
-07 (C): └───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
+07 (C): └─ <ENTER> to run, <Ctrl-c> to exit ────────────────────────────────────────────────────────────────────────────────────┘
 08 (C): ┌ command2 ─────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
 09 (C): │                                                                                                                       │
 10 (C): │                                              Waiting to run: command2                                                 │
diff --git a/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__swap_layouts_not_including_command_panes_present_in_existing_layout.snap b/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__swap_layouts_not_including_command_panes_present_in_existing_layout.snap
index dd0cbbd72..edac6ec4b 100644
--- a/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__swap_layouts_not_including_command_panes_present_in_existing_layout.snap
+++ b/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__swap_layouts_not_including_command_panes_present_in_existing_layout.snap
@@ -1,6 +1,5 @@
 ---
 source: zellij-server/src/tab/./unit/tab_integration_tests.rs
-assertion_line: 4904
 expression: snapshot
 ---
 00 (C): I am a                                                                                                                   
@@ -16,7 +15,7 @@ expression: snapshot
 10 (C): │                                              Waiting to run: command1                                                 │
 11 (C): │                                                                                                                       │
 12 (C): │                                          <ENTER> to run, <Ctrl-c> to exit                                             │
-13 (C): └───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
+13 (C): └─ <ENTER> to run, <Ctrl-c> to exit ────────────────────────────────────────────────────────────────────────────────────┘
 14 (C): ┌ command2 ─────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
 15 (C): │                                              Waiting to run: command2                                                 │
 16 (C): │                                                                                                                       │
diff --git a/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__swap_layouts_not_including_plugin_panes_present_in_existing_layout.snap b/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__swap_layouts_not_including_plugin_panes_present_in_existing_layout.snap
index 258ea58be..d5366c733 100644
--- a/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__swap_layouts_not_including_plugin_panes_present_in_existing_layout.snap
+++ b/zellij-server/src/tab/unit/snapshots/zellij_server__tab__tab_integration_tests__swap_layouts_not_including_plugin_panes_present_in_existing_layout.snap
@@ -1,6 +1,5 @@
 ---
 source: zellij-server/src/tab/./unit/tab_integration_tests.rs
-assertion_line: 5035
 expression: snapshot
 ---
 00 (C): ┌ Pane #2 ──────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
@@ -10,7 +9,7 @@ expression: snapshot
 04 (C): │                                              Waiting to run: command1                                                 │
 05 (C): │                                                                                                                       │
 06 (C): │                                          <ENTER> to run, <Ctrl-c> to exit                                             │
-07 (C): └───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
+07 (C): └─ <ENTER> to run, <Ctrl-c> to exit ────────────────────────────────────────────────────────────────────────────────────┘
 08 (C): ┌ command2 ─────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
 09 (C): │                                                                                                                       │
 10 (C): │                                              Waiting to run: command2                                                 │