Add interactive navigation to sidebar with click handlers

Co-authored-by: GZTimeWalker <28180262+GZTimeWalker@users.noreply.github.com>

Author: Copilot

crates/wsrx-desktop-gpui/src/views/root.rs

@@ -1,5 +1,5 @@
 // Root view - Main application window
-use gpui::{Context, Render, Window, div, prelude::*, Entity};
+use gpui::{Context, Render, Window, div, prelude::*, Entity, SharedString, WeakEntity};
 use crate::models::Page;
 use crate::styles::colors;
 use super::{SidebarView, GetStartedView, ConnectionsView, NetworkLogsView, SettingsView};
@@ -22,18 +22,38 @@ impl RootView {
     pub fn new(window: &mut Window, cx: &mut Context<Self>) -> Self {
         let current_page = Page::GetStarted;
 
-        Self {
+        let mut root = Self {
             current_page,
             sidebar: cx.new(|cx| SidebarView::new(window, cx, current_page)),
             get_started: cx.new(|cx| GetStartedView::new(window, cx)),
             connections: cx.new(|cx| ConnectionsView::new(window, cx)),
             network_logs: cx.new(|cx| NetworkLogsView::new(window, cx)),
             settings: cx.new(|cx| SettingsView::new(window, cx)),
-        }
+        };
+        
+        // Set up the navigation callback for sidebar
+        let weak_self = cx.weak_entity();
+        root.sidebar.update(cx, |sidebar, _| {
+            sidebar.set_on_page_change(Box::new(move |page, cx| {
+                if let Some(root) = weak_self.upgrade() {
+                    root.update(cx, |root, cx| {
+                        root.set_page(page, cx);
+                    });
+                }
+            }));
+        });
+        
+        root
     }
 
-    pub fn set_page(&mut self, page: Page, _cx: &mut Context<Self>) {
+    pub fn set_page(&mut self, page: Page, cx: &mut Context<Self>) {
         self.current_page = page;
+        // Update sidebar to reflect the new active page
+        self.sidebar.update(cx, |sidebar, cx| {
+            sidebar.set_active_page(page);
+            cx.notify();
+        });
+        cx.notify(); // Trigger re-render
     }
 
     fn render_sidebar(&self) -> impl IntoElement {

crates/wsrx-desktop-gpui/src/views/sidebar.rs

@@ -1,46 +1,69 @@
 // Sidebar view - Navigation sidebar
-use gpui::{Context, Render, Window, div, prelude::*};
+use gpui::{Context, Render, Window, div, prelude::*, App, SharedString};
 use crate::models::Page;
 use crate::styles::colors;
 
+type PageChangeCallback = Box<dyn Fn(Page, &mut App) + Send + Sync>;
+
 pub struct SidebarView {
     active_page: Page,
+    on_page_change: Option<PageChangeCallback>,
 }
 
 impl SidebarView {
     pub fn new(_window: &mut Window, _cx: &mut Context<Self>, active_page: Page) -> Self {
-        Self { active_page }
+        Self { 
+            active_page,
+            on_page_change: None,
+        }
+    }
+    
+    pub fn set_on_page_change(&mut self, callback: PageChangeCallback) {
+        self.on_page_change = Some(callback);
+    }
+    
+    pub fn set_active_page(&mut self, page: Page) {
+        self.active_page = page;
     }
 
-    fn render_tab(&self, page: Page, label: impl Into<String>) -> impl IntoElement {
+    fn render_tab(&self, page: Page, label: impl Into<String>, cx: &mut Context<Self>) -> impl IntoElement {
         let is_active = self.active_page == page;
         let label_text = label.into();
+        let id = SharedString::from(format!("sidebar-tab-{:?}", page));
+        
         div()
+            .id(id)
             .flex()
             .items_center()
             .px_4()
             .py_3()
+            .cursor_pointer()
             .when(is_active, |div| {
                 div.bg(colors::accent())
             })
             .when(!is_active, |div| {
                 div.hover(|div| div.bg(gpui::rgba(0x00000030)))
             })
+            .on_click(cx.listener(move |this, _event, _window, cx| {
+                if let Some(ref callback) = this.on_page_change {
+                    callback(page, cx);
+                }
+            }))
             .child(label_text)
     }
 }
 
 impl Render for SidebarView {
-    fn render(&mut self, _window: &mut Window, _cx: &mut Context<Self>) -> impl IntoElement {
+    fn render(&mut self, _window: &mut Window, cx: &mut Context<Self>) -> impl IntoElement {
         div()
             .flex()
             .flex_col()
             .gap_1()
             .p_2()
-            .child(self.render_tab(Page::GetStarted, "Get Started"))
-            .child(self.render_tab(Page::Connections, "Connections"))
-            .child(self.render_tab(Page::NetworkLogs, "Network Logs"))
-            .child(self.render_tab(Page::Settings, "Settings"))
+            .child(self.render_tab(Page::GetStarted, "Get Started", cx))
+            .child(self.render_tab(Page::Connections, "Connections", cx))
+            .child(self.render_tab(Page::NetworkLogs, "Network Logs", cx))
+            .child(self.render_tab(Page::Settings, "Settings", cx))
     }
 }