Fix broken button functionality by correcting layout logic

The layout redesign introduced a bug where buttons stopped working when a
channel was selected. The issue was that the code always used a single-column
layout for channel views, but when router views (Accounts, Relays, etc.) were
triggered by button clicks, only column 0 was rendered instead of all columns.

Changes:
- Check router state upfront to determine the correct layout
- Only use channel layout when channel is selected AND no router view is active
- Use normal multi-column layout when router views are shown
- Remove debug println statements
- Remove unused accounts field from DesktopSidePanel

This ensures all button clicks are properly handled and render the appropriate
views across all columns.

crates/notedeck_columns/src/app.rs

@@ -1155,10 +1155,17 @@ fn timelines_view(
 
     let mut can_take_drag_from = Vec::new();
 
-    // Check if a channel is selected to determine layout
-    let is_channel_selected = app.channels_cache.active_channels(ctx.accounts).selected_channel().is_some();
+    // Check if a channel is selected AND we should show it (not overridden by router views)
+    let has_selected_channel = app.channels_cache.active_channels(ctx.accounts).selected_channel().is_some();
+    let router = get_active_columns_mut(ctx.i18n, ctx.accounts, &mut app.decks_cache).get_selected_router();
+    let has_router_view = router.routes().iter().any(|route| {
+        matches!(route, Route::Accounts(_) | Route::Relays | Route::AddColumn(_) | Route::ComposeNote | Route::Search | Route::NewDeck | Route::EditDeck(_))
+    });
 
-    if is_channel_selected {
+    // Only use channel layout if a channel is selected AND we're not showing a router view
+    let show_channel_layout = has_selected_channel && !has_router_view;
+
+    if show_channel_layout {
         // When channel is selected: Side Panel | Channel Sidebar | Content Area (remainder)
         StripBuilder::new(ui)
             .size(Size::exact(ui::side_panel::SIDE_PANEL_WIDTH))
@@ -1172,7 +1179,6 @@ fn timelines_view(
                 let side_panel = DesktopSidePanel::new(
                     ctx.accounts.get_selected_account(),
                     &app.decks_cache,
-                    ctx.accounts,
                     ctx.i18n,
                 )
                 .show(ui);
@@ -1227,49 +1233,13 @@ fn timelines_view(
                 );
             });
 
-            // Check if a channel is selected and if we need to show other views first
+            // Render ChatView for the selected channel
             let selected_timeline_kind = app.channels_cache
                 .active_channels(ctx.accounts)
                 .selected_channel()
                 .map(|c| c.timeline_kind.clone());
 
-            // Check if we need to show other views (Accounts, Relays, etc.) instead of channel
-            let router = get_active_columns_mut(ctx.i18n, ctx.accounts, &mut app.decks_cache).get_selected_router();
-            let should_override_channel = router.routes().iter().any(|route| {
-                matches!(route, Route::Accounts(_) | Route::Relays | Route::AddColumn(_) | Route::ComposeNote | Route::Search | Route::NewDeck | Route::EditDeck(_))
-            });
-
-            // Debug: Check the override condition
-            if should_override_channel {
-                println!("DEBUG: Overriding channel view, routes count: {}", router.routes().len());
-            } else {
-                println!("DEBUG: Not overriding channel view, routes count: {}", router.routes().len());
-            }
-
-            if should_override_channel {
-                // Render router-based views (Accounts, Relays, etc.)
-                strip.cell(|ui| {
-                    let rect = ui.available_rect_before_wrap();
-                    let v_line_stroke = ui.visuals().widgets.noninteractive.bg_stroke;
-                    let inner_rect = {
-                        let mut inner = rect;
-                        inner.set_right(rect.right() - v_line_stroke.width);
-                        inner
-                    };
-
-                    // Render navigation for the router views
-                    println!("DEBUG: Calling nav::render_nav for column 0");
-                    let resp = nav::render_nav(0, inner_rect, app, ctx, ui);
-                    println!("DEBUG: nav::render_nav completed, responses: {}", responses.len());
-                    can_take_drag_from.extend(resp.can_take_drag_from());
-                    responses.push(resp);
-
-                    // vertical line
-                    ui.painter()
-                        .vline(rect.right(), rect.y_range(), v_line_stroke);
-                });
-            } else if let Some(timeline_kind) = selected_timeline_kind {
-                // Render ChatView for the selected channel
+            if let Some(timeline_kind) = selected_timeline_kind {
                 strip.cell(|ui| {
                     let rect = ui.available_rect_before_wrap();
                     let v_line_stroke = ui.visuals().widgets.noninteractive.bg_stroke;
@@ -1353,7 +1323,6 @@ fn timelines_view(
                     let side_panel = DesktopSidePanel::new(
                         ctx.accounts.get_selected_account(),
                         &app.decks_cache,
-                        ctx.accounts,
                         ctx.i18n,
                     )
                     .show(ui);

crates/notedeck_columns/src/ui/side_panel.rs

@@ -27,7 +27,6 @@ static ICON_WIDTH: f32 = 40.0;
 pub struct DesktopSidePanel<'a> {
     selected_account: &'a UserAccount,
     decks_cache: &'a DecksCache,
-    accounts: &'a Accounts,
     i18n: &'a mut Localization,
 }
 
@@ -66,13 +65,11 @@ impl<'a> DesktopSidePanel<'a> {
     pub fn new(
         selected_account: &'a UserAccount,
         decks_cache: &'a DecksCache,
-        accounts: &'a Accounts,
         i18n: &'a mut Localization,
     ) -> Self {
         Self {
             selected_account,
             decks_cache,
-            accounts,
             i18n,
         }
     }