fix: interactive docs and 256-color themes

Author: nicholasjpaterno

CHANGELOG.md

@@ -5,6 +5,36 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.3.1] - 2025-11-21
+
+### Fixed
+
+- **Theme Color Mode Application** - Improved theme initialization order for more consistent color rendering
+  - Color mode now applied before custom colors for better accuracy
+  - Theme-specific 256-color variants now correctly loaded based on color mode
+  - Fixes color inconsistencies when using custom theme colors with 256-color fallback
+
+### Changed
+
+- **Code Formatting** - Applied consistent code formatting throughout codebase
+- **Documentation** - Enhanced README with more detailed interactive mode instructions
+  - Added comprehensive keyboard shortcuts for interactive mode
+  - Clarified table navigation and editing workflow
+  - Better organization of feature descriptions
+
+### Technical
+
+- **Theme Architecture** (`src/tui/theme.rs`)
+  - Added `from_name_256()` method to load theme-specific 256-color variants
+  - Reordered `with_color_mode()` to apply before `with_custom_colors()`
+  - Each theme now has dedicated 256-color palette matching official theme colors
+  - Better color accuracy in 256-color terminals
+
+- **App State** (`src/tui/app.rs`)
+  - Updated theme initialization: `from_name() → with_color_mode() → with_custom_colors()`
+  - Custom colors now properly respect color mode constraints
+  - Improved formatting consistency
+
 ## [0.3.0] - 2025-11-20
 
 ### Added

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "treemd"
-version = "0.3.0"
+version = "0.3.1"
 edition = "2024"
 authors = ["epistates, inc. <nick@epistates.com>"]
 description = "A markdown navigator with tree-based structural navigation and syntax highlighting"

Cargo.toml

@@ -18,6 +18,9 @@ A markdown navigator with tree-based structural navigation. Like `tree`, but int
 ### Interactive TUI
 
 - **Dual-pane interface** - Navigate outline while viewing content
+- **Interactive mode** - Navigate, edit, and interact with all markdown elements (tables, checkboxes, links, code blocks)
+- **Table navigation & editing** - Navigate cells with vim keys (hjkl), edit cell content in-place, copy cells/rows/tables
+- **Checkbox toggling** - Toggle task list items with instant file updates
 - **Live editing** - Edit files in default editor with auto-reload (respects $VISUAL/$EDITOR)
 - **Link following** - Follow markdown links with visual popup, supports anchor/file/wikilink/external URLs
 - **Navigation history** - Back/forward between files with full state preservation
@@ -27,7 +30,7 @@ A markdown navigator with tree-based structural navigation. Like `tree`, but int
 - **Collapsible tree** - Expand/collapse sections with Space/Enter
 - **Bookmarks** - Mark positions (`m`) and jump back (`'`)
 - **Adjustable layout** - Toggle outline visibility, resize panes
-- **Rich rendering** - Bold, italic, inline code, lists, blockquotes, code blocks
+- **Rich rendering** - Bold, italic, inline code, lists, blockquotes, code blocks, tables with box-drawing characters
 
 ### CLI Mode
 
@@ -106,6 +109,23 @@ treemd README.md
 - `Shift+F` - Go forward in navigation history
 - `Esc` - Exit link follow mode
 
+*Interactive Mode:*
+- `i` - Enter interactive mode (navigate all interactive elements)
+- `Tab`/`j`/`k` or `↓/↑` - Navigate between elements
+- `Enter` - Activate element (toggle checkbox, follow link, enter table mode)
+- `Space` - Toggle checkboxes or details blocks
+- `y` - Copy element content (code blocks, table cells, links)
+- `Esc` - Exit interactive mode
+
+*Table Navigation (in interactive mode):*
+- `Enter` on table - Enter table navigation mode
+- `h`/`j`/`k`/`l` or arrow keys - Navigate table cells
+- `y` - Copy current cell content
+- `Y` - Copy current row (tab-separated)
+- `r` - Copy entire table as markdown
+- `Enter` on cell - Edit cell content
+- `Esc` - Exit table navigation mode
+
 *Editing & System:*
 - `e` - Edit current file in default editor (respects $VISUAL or $EDITOR)
 - `t` - Cycle color theme
@@ -119,6 +139,9 @@ treemd README.md
 
 **Interface Features:**
 - **Live editing** - Edit files in your default editor and auto-reload (press `e`)
+- **Interactive mode** - Navigate and interact with all markdown elements (press `i`)
+- **Table editing** - Navigate cells with vim keys, edit cell content in-place
+- **Checkbox toggling** - Toggle task list items and save changes to file
 - **Link following popup** - Visual navigator shows all links with highlighting (press `f`)
 - **Multi-file navigation** - Load files via links with full history (back/forward)
 - **External URL opening** - Opens links in default browser automatically

README.md

@@ -4,7 +4,7 @@ use serde::{Deserialize, Serialize};
 use std::fs;
 use std::path::PathBuf;
 
-#[derive(Debug, Clone, Serialize, Deserialize)]
+#[derive(Debug, Clone, Default, Serialize, Deserialize)]
 pub struct Config {
     #[serde(default)]
     pub ui: UiConfig,
@@ -122,15 +122,6 @@ impl ColorValue {
     }
 }
 
-impl Default for Config {
-    fn default() -> Self {
-        Self {
-            ui: UiConfig::default(),
-            terminal: TerminalConfig::default(),
-            theme: CustomThemeConfig::default(),
-        }
-    }
-}
 
 impl Default for UiConfig {
     fn default() -> Self {

src/config.rs

@@ -115,11 +115,11 @@ impl App {
 
         let content_lines = document.content.lines().count();
 
-        // Load theme from config, apply custom colors, then apply color mode
+        // Load theme from config, apply color mode, then apply custom colors
         let current_theme = config.theme_name();
         let theme = Theme::from_name(current_theme)
-            .with_custom_colors(&config.theme)
-            .with_color_mode(color_mode);
+            .with_color_mode(color_mode, current_theme)
+            .with_custom_colors(&config.theme, color_mode);
 
         // Load outline width from config
         let outline_width = config.ui.outline_width;
@@ -440,7 +440,8 @@ impl App {
             // Element is below viewport - scroll down to show it
             else if end > viewport_end {
                 // Try to position element at top of viewport
-                self.content_scroll = start.min(self.content_height.saturating_sub(viewport_height));
+                self.content_scroll =
+                    start.min(self.content_height.saturating_sub(viewport_height));
             }
             // Element partially visible at bottom - ensure fully visible
             else if start >= scroll && end > viewport_end {
@@ -677,8 +678,8 @@ impl App {
         self.current_theme = new_theme;
         // Apply color mode when setting theme (also apply custom colors from config)
         self.theme = Theme::from_name(new_theme)
-            .with_custom_colors(&self.config.theme)
-            .with_color_mode(self.color_mode);
+            .with_color_mode(self.color_mode, new_theme)
+            .with_custom_colors(&self.config.theme, self.color_mode);
         self.show_theme_picker = false;
 
         // Save to config (silently ignore errors)
@@ -1140,7 +1141,7 @@ impl App {
         // Get current section content to index
         let content = if let Some(selected) = self.selected_heading_text() {
             self.document
-                .extract_section(&selected)
+                .extract_section(selected)
                 .unwrap_or_else(|| self.document.content.clone())
         } else {
             self.document.content.clone()
@@ -1234,10 +1235,9 @@ impl App {
             }
             ElementType::Table { rows, cols, .. } => {
                 // Enter table navigation mode
-                if let Err(e) = self.interactive_state.enter_table_mode() {
-                    return Err(e);
-                }
-                self.status_message = Some(self.interactive_state.table_status_text(rows + 1, *cols));
+                self.interactive_state.enter_table_mode()?;
+                self.status_message =
+                    Some(self.interactive_state.table_status_text(rows + 1, *cols));
                 Ok(())
             }
         }
@@ -1247,7 +1247,7 @@ impl App {
     fn reindex_interactive_elements(&mut self) {
         let content = if let Some(selected) = self.selected_heading_text() {
             self.document
-                .extract_section(&selected)
+                .extract_section(selected)
                 .unwrap_or_else(|| self.document.content.clone())
         } else {
             self.document.content.clone()
@@ -1268,30 +1268,25 @@ impl App {
         // Get the checkbox content text to use as identifier
         let checkbox_content = {
             let content = if let Some(selected) = self.selected_heading_text() {
-                self.document.extract_section(&selected).unwrap_or_else(|| self.document.content.clone())
+                self.document
+                    .extract_section(selected)
+                    .unwrap_or_else(|| self.document.content.clone())
             } else {
                 self.document.content.clone()
             };
 
             use crate::parser::content::parse_content;
             let blocks = parse_content(&content, 0);
 
-            if let Some(block) = blocks.get(block_idx) {
-                if let crate::parser::output::Block::List { items, .. } = block {
-                    if let Some(item) = items.get(item_idx) {
-                        Some(item.content.clone())
-                    } else {
-                        None
-                    }
-                } else {
-                    None
-                }
+            if let Some(crate::parser::output::Block::List { items, .. }) = blocks.get(block_idx) {
+                items.get(item_idx).map(|item| item.content.clone())
             } else {
                 None
             }
         };
 
-        let checkbox_content = checkbox_content.ok_or_else(|| "Could not find checkbox content".to_string())?;
+        let checkbox_content =
+            checkbox_content.ok_or_else(|| "Could not find checkbox content".to_string())?;
 
         // Read the current file
         let file_content = std::fs::read_to_string(&self.current_file_path)
@@ -1340,7 +1335,9 @@ impl App {
             let trimmed = line.trim_start();
 
             // Check if this is a checkbox line
-            if (trimmed.starts_with("- [ ]") || trimmed.starts_with("- [x]") || trimmed.starts_with("- [X]"))
+            if (trimmed.starts_with("- [ ]")
+                || trimmed.starts_with("- [x]")
+                || trimmed.starts_with("- [X]"))
                 && !found
             {
                 // Extract the text after the checkbox marker
@@ -1435,21 +1432,23 @@ impl App {
     /// Get table data for current interactive element
     fn get_current_table_data(&self) -> Option<(Vec<String>, Vec<Vec<String>>)> {
         if let Some(element) = self.interactive_state.current_element() {
-            if let crate::tui::interactive::ElementType::Table { block_idx, .. } = &element.element_type {
+            if let crate::tui::interactive::ElementType::Table { block_idx, .. } =
+                &element.element_type
+            {
                 // Parse current section to get table data
                 let content = if let Some(selected) = self.selected_heading_text() {
-                    self.document.extract_section(&selected).unwrap_or_else(|| self.document.content.clone())
+                    self.document
+                        .extract_section(selected)
+                        .unwrap_or_else(|| self.document.content.clone())
                 } else {
                     self.document.content.clone()
                 };
 
                 use crate::parser::content::parse_content;
                 let blocks = parse_content(&content, 0);
 
-                if let Some(block) = blocks.get(*block_idx) {
-                    if let crate::parser::output::Block::Table { headers, rows, .. } = block {
-                        return Some((headers.clone(), rows.clone()));
-                    }
+                if let Some(crate::parser::output::Block::Table { headers, rows, .. }) = blocks.get(*block_idx) {
+                    return Some((headers.clone(), rows.clone()));
                 }
             }
         }
@@ -1592,11 +1591,9 @@ impl App {
         if let Some(element) = self.interactive_state.current_element() {
             let block_idx = element.id.block_idx;
 
-            if let Some(block) = blocks.get(block_idx) {
-                if let crate::parser::output::Block::Table { .. } = block {
-                    // Find this table in the full file content
-                    return self.replace_table_cell_in_file(content, row, col, new_value);
-                }
+            if let Some(crate::parser::output::Block::Table { .. }) = blocks.get(block_idx) {
+                // Find this table in the full file content
+                return self.replace_table_cell_in_file(content, row, col, new_value);
             }
         }