hyperlist 1.9.1 → 1.9.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 485f51c3f998ca45514979e831f928a2ca907674483ba1ceef644ce6db22423e
-  data.tar.gz: ca21fe8db31f2b1ac1a1e8059c92f94dbc17a8d42790247238f8073c77ef973d
+  metadata.gz: 56e8ead48066f1f09b28d2e014b5c8f1bc6576dffda4b45a5975cde75721399a
+  data.tar.gz: 8fe7e8ed277872310b4d0230e45e80c7ac886cbb458244ae6016f48858c661d8
 SHA512:
-  metadata.gz: d40e130fbff4cb2564f8a9c541257cd084072f1917067d529a86427c63b0e3716b7a6704f584cf64dc3e204fd0c63a71a20443f3bcef99bc5377bb43c657b7f3
-  data.tar.gz: 5e6a7cdf9f8b2fb7e93382e41ba8d6e2e39d0403a0e64480d763d8385f5e4d7320684ea4b1fd7503d5a4ed08fe515d5176a04d8ab8723f9e43b7ab9d04a9e2b8
+  metadata.gz: 11d7b1fce3b5179c2cdf1278deb4e2e0fafb74aa36f77ea8a7dd4ab494b53bc2af79942769390cb82a8d9a392b830947ec5bdd2b991462d27f53a148ff42809c
+  data.tar.gz: 36f631cbc904a3fb4fd28b9e3c17cbd83f55054d04e5add1a2d58ba86a092315a42976ed1eddfedd264fe90f5b54a4069f26e2e9bce3faa1959c0bf2082a28c2

data/CHANGELOG.md

@@ -2,6 +2,25 @@
 
 All notable changes to the HyperList Ruby TUI will be documented in this file.
 
+## [1.9.3] - 2025-12-02
+
+### Fixed
+- **Help/documentation viewer navigation**
+  - Fixed invisible cursor on blank separator lines in help (?) and documentation (??) viewers
+  - Blank lines now contain a space so cursor highlighting is visible when navigating
+- **Presentation mode state preservation**
+  - Help, documentation, and recent files viewers now properly save/restore presentation mode state
+  - Prevents presentation mode logic from interfering with viewer navigation
+
+## [1.9.2] - 2025-12-02
+
+### Fixed
+- **Presentation mode completely rewritten**
+  - Now dynamically folds/unfolds as you navigate (like hyperlist.vim)
+  - Entering presentation mode (C-P) now stays on the current item instead of jumping to the first line
+  - Moving up/down properly reveals the path to the current item while folding everything else
+  - Uses proper item tracking via real indices to handle fold state changes correctly
+
 ## [1.9.1] - 2025-12-02
 
 ### Added

data/README.md

@@ -29,7 +29,20 @@ For historical context and the original VIM implementation, see: [hyperlist.vim]
 ### Help Screen
 ![HyperList Help](img/screenshot_help.png)
 
-## What's New in v1.9.1
+## What's New in v1.9.3
+
+### Help Viewer Navigation Fixed
+- Fixed invisible cursor on blank separator lines in help (?) and documentation (??) viewers
+- Presentation mode state now properly preserved when entering/exiting viewers
+
+## Previous Release: v1.9.2
+
+### Presentation Mode Fixed
+- **Dynamic folding**: Presentation mode now works like hyperlist.vim - folds/unfolds dynamically as you navigate
+- **Cursor preservation**: Entering presentation mode (C-P) stays on the current item instead of jumping to the first line
+- **Proper navigation**: Moving up/down reveals the path to the current item while folding everything else
+
+## Previous Release: v1.9.1
 
 ### External File Change Detection
 - **Auto-detect external changes**: HyperList now monitors when the file is modified by other processes (vim, another Claude Code session, etc.)

data/hyperlist

@@ -7,7 +7,7 @@
 # Check for help/version BEFORE loading any libraries
 if ARGV[0] == '-h' || ARGV[0] == '--help'
   puts <<~HELP
-    HyperList v1.9.1 - Terminal User Interface for HyperList files
+    HyperList v1.9.3 - Terminal User Interface for HyperList files
     
     USAGE
       hyperlist [OPTIONS] [FILE]
@@ -52,7 +52,7 @@ if ARGV[0] == '-h' || ARGV[0] == '--help'
   HELP
   exit 0
 elsif ARGV[0] == '-v' || ARGV[0] == '--version'
-  puts "HyperList v1.9.1"
+  puts "HyperList v1.9.3"
   exit 0
 end
 
@@ -73,7 +73,7 @@ class HyperListApp
   include Rcurses::Input
   include Rcurses::Cursor
   
-  VERSION = "1.9.1"
+  VERSION = "1.9.3"
   
   def initialize(filename = nil)
     @filename = filename ? File.expand_path(filename) : nil
@@ -596,7 +596,11 @@ class HyperListApp
     saved_offset = @offset
     saved_filename = @filename
     saved_modified = @modified
-    
+    saved_presentation_mode = @presentation_mode
+
+    # Disable presentation mode for recent files viewer
+    @presentation_mode = false
+
     # Create items for recent files display
     @items = []
     @items << {"text" => "RECENT FILES (press Enter to open, q to cancel)", "level" => 0, "fold" => false, "raw" => true}
@@ -627,6 +631,7 @@ class HyperListApp
         @offset = saved_offset
         @filename = saved_filename
         @modified = saved_modified
+        @presentation_mode = saved_presentation_mode
         break
       when "j", "DOWN"
         move_down if @current < @items.length - 1
@@ -641,9 +646,11 @@ class HyperListApp
             @offset = saved_offset
             @filename = saved_filename
             @modified = saved_modified
+            @presentation_mode = saved_presentation_mode
             @message = "Unsaved changes! Save first before opening another file"
             break
           else
+            @presentation_mode = saved_presentation_mode
             @filename = File.expand_path(selected_file)
             load_file(@filename)
             @current = 0
@@ -663,9 +670,11 @@ class HyperListApp
             @offset = saved_offset
             @filename = saved_filename
             @modified = saved_modified
+            @presentation_mode = saved_presentation_mode
             @message = "Unsaved changes! Save first before opening another file"
             break
           else
+            @presentation_mode = saved_presentation_mode
             @filename = File.expand_path(selected_file)
             load_file(@filename)
             @current = 0
@@ -677,7 +686,7 @@ class HyperListApp
       end
     end
   end
-  
+
   def save_file
     return unless @filename
     
@@ -2043,28 +2052,28 @@ class HyperListApp
 
   def move_up
     if @presentation_mode
-      # In presentation mode, we need to handle navigation differently
-      visible_before = get_visible_items
-      
+      # In presentation mode: move to target, then update folds
+      visible = get_visible_items
+      return if visible.empty?
+
+      # Calculate target position
       if @current == 0
-        # Wrap around to last item
-        target_index = visible_before.length - 1
+        target_index = visible.length - 1
       else
         target_index = @current - 1
       end
-      
-      # Get the actual item we want to move to
-      target_item = visible_before[target_index]
-      target_real_idx = get_real_index(target_item)
-      
-      # Update presentation focus for the target item
+
+      # Get the real index of the target item
+      target_real_idx = get_real_index(visible[target_index])
+
+      # Temporarily set current to target so update_presentation_focus works on it
       @current = target_index
       update_presentation_focus
-      
-      # Now find where the target item ended up after reorganization
-      visible_after = get_visible_items
-      visible_after.each_with_index do |item, idx|
-        if get_real_index(item) == target_real_idx
+
+      # After fold changes, find where the target item is now
+      new_visible = get_visible_items
+      new_visible.each_with_index do |item, idx|
+        if item["_real_index"] == target_real_idx
           @current = idx
           break
         end
@@ -2072,9 +2081,8 @@ class HyperListApp
     else
       # Normal mode navigation
       max_items = get_visible_items.length - 1
-      
+
       if @current == 0
-        # Wrap around to last item
         @current = max_items
       else
         @current = [@current - 1, 0].max
@@ -2084,29 +2092,29 @@ class HyperListApp
   
   def move_down
     if @presentation_mode
-      # In presentation mode, we need to handle navigation differently
-      visible_before = get_visible_items
-      max = visible_before.length - 1
-      
-      if @current == max
-        # Wrap around to first item
+      # In presentation mode: move to target, then update folds
+      visible = get_visible_items
+      return if visible.empty?
+      max = visible.length - 1
+
+      # Calculate target position
+      if @current >= max
         target_index = 0
       else
         target_index = @current + 1
       end
-      
-      # Get the actual item we want to move to
-      target_item = visible_before[target_index]
-      target_real_idx = get_real_index(target_item)
-      
-      # Update presentation focus for the target item
+
+      # Get the real index of the target item
+      target_real_idx = get_real_index(visible[target_index])
+
+      # Temporarily set current to target so update_presentation_focus works on it
       @current = target_index
       update_presentation_focus
-      
-      # Now find where the target item ended up after reorganization
-      visible_after = get_visible_items
-      visible_after.each_with_index do |item, idx|
-        if get_real_index(item) == target_real_idx
+
+      # After fold changes, find where the target item is now
+      new_visible = get_visible_items
+      new_visible.each_with_index do |item, idx|
+        if item["_real_index"] == target_real_idx
           @current = idx
           break
         end
@@ -2114,9 +2122,8 @@ class HyperListApp
     else
       # Normal mode navigation
       max = get_visible_items.length - 1
-      
-      if @current == max
-        # Wrap around to first item
+
+      if @current >= max
         @current = 0
       else
         @current = [@current + 1, max].min
@@ -2345,84 +2352,80 @@ class HyperListApp
       # Enter presentation mode
       # Remember which item we're on before any folding changes
       visible_items = get_visible_items
-      if @current < visible_items.length
-        target_item = visible_items[@current]
-      end
-      
+      return if visible_items.empty? || @current >= visible_items.length
+
+      # Get the real index of the current item
+      target_real_idx = visible_items[@current]["_real_index"]
+
       @presentation_mode = true
-      # Clear cache to force re-rendering  
+      # Clear cache to force re-rendering
       @processed_cache.clear
       update_presentation_focus
-      
-      # Make sure cursor is still on the same item after initial folding
-      if target_item
-        new_visible = get_visible_items
-        new_position = new_visible.index(target_item)
-        @current = new_position if new_position
+
+      # Find where the target item is after fold changes
+      new_visible = get_visible_items
+      new_visible.each_with_index do |item, idx|
+        if item["_real_index"] == target_real_idx
+          @current = idx
+          break
+        end
       end
-      
-      @message = "Presentation mode enabled - focus on current item"
+
+      @message = "Presentation mode enabled"
     end
   end
   
   def update_presentation_focus
-    # This method updates the focus in presentation mode
-    # It will be called whenever the cursor moves
+    # Presentation mode: fold everything, then reveal path to current item
+    # Like vim's: fold to level 0, then zv (reveal current line)
     return unless @presentation_mode
-    
+
     visible_items = get_visible_items
     return if visible_items.empty? || @current >= visible_items.length
-    
-    # Remember which item we're focused on
+
+    # Get the current item before we change anything
     current_item = visible_items[@current]
-    current_level = current_item["level"]
     current_real_idx = get_real_index(current_item)
-    
-    # First, fold everything
+    return unless current_real_idx
+
+    current_level = current_item["level"]
+
+    # Step 1: Fold everything that has children
     @items.each_with_index do |item, idx|
       item["fold"] = has_children?(idx, @items)
       item["presentation_focus"] = false
     end
-    
-    # Mark current item as in focus and unfold it in the main items array
-    if current_real_idx && current_real_idx < @items.length
-      @items[current_real_idx]["presentation_focus"] = true
-      @items[current_real_idx]["fold"] = false
-    end
-    
-    # Unfold all ancestors of current item
-    ancestor_indices = []
+
+    # Step 2: Unfold all ancestors (path from root to current item)
     search_level = current_level - 1
     idx = current_real_idx - 1
-    
     while idx >= 0 && search_level >= 0
       if @items[idx]["level"] == search_level
-        @items[idx]["fold"] = false
-        @items[idx]["presentation_focus"] = false  # Ancestors visible but not focused
-        ancestor_indices << idx
+        @items[idx]["fold"] = false  # Unfold this ancestor
         search_level -= 1
       end
       idx -= 1
     end
-    
-    # Mark immediate children as in focus (only one level down)
+
+    # Step 3: Unfold the current item to show its direct children
+    @items[current_real_idx]["fold"] = false
+    @items[current_real_idx]["presentation_focus"] = true
+
+    # Step 4: Mark direct children as in focus (for visual highlighting)
     idx = current_real_idx + 1
     while idx < @items.length && @items[idx]["level"] > current_level
       if @items[idx]["level"] == current_level + 1
         @items[idx]["presentation_focus"] = true
-        # Don't unfold children - let them stay folded unless user explicitly unfolds
       end
       idx += 1
     end
-    
-    # Now recalculate the cursor position to point to the same item
+
+    # Step 5: Recalculate cursor position after fold changes
     new_visible_items = get_visible_items
     new_position = new_visible_items.index(current_item)
-    if new_position
-      @current = new_position
-    end
-    
-    # Clear cache to force re-rendering with new focus
+    @current = new_position if new_position
+
+    # Clear cache to force re-rendering
     @processed_cache.clear
   end
   
@@ -3744,9 +3747,9 @@ class HyperListApp
     # Build help text using consistent formatting
     help_lines = []
     help_lines << " Press #{"?".fg("10")} for full documentation, #{"UP/DOWN".fg("10")} to scroll, or any other key to return"
-    help_lines << ""
+    help_lines << " "
     help_lines << "#{"HYPERLIST KEY BINDINGS".b}"
-    help_lines << ""
+    help_lines << " "
     help_lines << "#{"NAVIGATION".fg("14")}"
     help_lines << help_line("#{"j/↓".fg("10")}", "Move down", "#{"k/↑".fg("10")}", "Move up")
     help_lines << help_line("#{"h".fg("10")}", "Go to parent", "#{"l".fg("10")}", "Go to first child")
@@ -3755,10 +3758,10 @@ class HyperListApp
     help_lines << help_line("#{"R".fg("10")}", "Jump to reference", "#{"F".fg("10")}", "Open file/URL")
     help_lines << help_line("#{"ma".fg("10")}", "Set mark 'a'", "#{"'a".fg("10")}", "Jump to mark 'a'")
     help_lines << help_line("#{"''".fg("10")}", "Jump to prev position", "#{"N".fg("10")}", "Next = template marker")
-    help_lines << ""
+    help_lines << " "
     help_lines << "#{"SEARCH".fg("14")}"
     help_lines << help_line("#{"/".fg("10")}", "Search forward", "#{"n".fg("10")}", "Next match")
-    help_lines << ""
+    help_lines << " "
     help_lines << "#{"FOLDING".fg("14")}"
     help_lines << help_line("#{"Space".fg("10")}", "Toggle fold", "#{"za".fg("10")}", "Toggle all folds")
     help_lines << help_line("#{"zo".fg("10")}", "Open fold", "#{"zc".fg("10")}", "Close fold")
@@ -3766,7 +3769,7 @@ class HyperListApp
     help_lines << help_line("#{"1-9".fg("10")}", "Expand to level", "#{"0".fg("10")}", "Multi-digit fold level")
     help_lines << help_line("#{"▶".fg("245")}", "Collapsed (hidden)")
     help_lines << help_line("#{"▷".fg("245")}", "Expanded (visible)")
-    help_lines << ""
+    help_lines << " "
     help_lines << "#{"EDITING".fg("14")}"
     help_lines << help_line("#{"i/Enter".fg("10")}", "Edit line", "#{"o".fg("10")}", "Insert line below")
     help_lines << help_line("#{"O".fg("10")}", "Insert line above", "#{"a".fg("10")}", "Insert child")
@@ -3782,7 +3785,7 @@ class HyperListApp
     help_lines << help_line("#{"C-UP".fg("10")}", "Move up in visible list", "#{"C-DOWN".fg("10")}", "Move down in visible list")
     help_lines << help_line("#{"→".fg("10")}", "Uncollapse item", "#{"←".fg("10")}", "Collapse item")
     help_lines << help_line("#{"Tab".fg("10")}", "Indent item+kids", "#{"S-Tab".fg("10")}", "Unindent item+kids")
-    help_lines << ""
+    help_lines << " "
     help_lines << "#{"SPECIAL FEATURES".fg("14")}"
     help_lines << help_line("#{"v".fg("10")}", "Toggle checkbox", "#{"V".fg("10")}", "Checkbox with date")
     help_lines << help_line("#{"C-X".fg("10")}", "Remove checkbox", "", "")
@@ -3790,7 +3793,7 @@ class HyperListApp
     help_lines << help_line("#{"C-P".fg("10")}", "Presentation mode", "#{"Tab/S-Tab".fg("10")}", "Next/prev sibling (in P)")
     help_lines << help_line("#{"Ma".fg("10")}", "Record macro 'a'", "#{"@a".fg("10")}", "Play macro 'a'")
     help_lines << help_line("#{"w".fg("10")}", "Switch panes (split view)")
-    help_lines << ""
+    help_lines << " "
     help_lines << "#{"FILE OPERATIONS".fg("14")}"
     help_lines << help_line("#{":w".fg("10")}", "Save", "#{":q".fg("10")}", "Quit")
     help_lines << help_line("#{":wq".fg("10")}", "Save and quit", "#{":e file".fg("10")}", "Open file")
@@ -3798,27 +3801,24 @@ class HyperListApp
     help_lines << help_line("#{":graph :g".fg("10")}", "Export to PNG graph", "#{":vsplit :vs".fg("10")}", "Split view")
     help_lines << help_line("#{":as on".fg("10")}", "Enable autosave", "#{":as off".fg("10")}", "Disable autosave")
     help_lines << help_line("#{":as N".fg("10")}", "Set interval (secs)", "#{":as".fg("10")}", "Show autosave status")
-    help_lines << ""
+    help_lines << " "
     help_lines << "#{"TAGGING & BATCH OPS".fg("14")}"
     help_lines << help_line("#{"t".fg("10")}", "Tag/untag item", "#{"u".fg("10")}", "Clear all tags")
     help_lines << help_line("#{"C-T".fg("10")}", "Tag by regex pattern", "", "")
     help_lines << help_line("#{"[T:N]".fg("245")}", "Status shows N tagged", "#{"Blue bg".fg("245")}", "Tagged items")
     help_lines << help_line("#{"D/y/Tab".fg("245")}", "Ops work on tagged items")
-    help_lines << ""
-
+    help_lines << " "
     help_lines << "#{"TEMPLATES".fg("14")}"
     help_lines << help_line("#{"T".fg("10")}", "Insert template", "#{":st".fg("10")}", "Save as template")
     help_lines << help_line("#{":dt".fg("10")}", "Delete template", "#{":lt".fg("10")}", "List user templates")
-    help_lines << ""
-    
+    help_lines << " "
     help_lines << "#{"CONFIGURATION".fg("14")}"
     help_lines << help_line("#{":set".fg("10")}", "Show all settings")
-    help_lines << help_line("#{":set o".fg("10")}", "Show option value")  
+    help_lines << help_line("#{":set o".fg("10")}", "Show option value")
     help_lines << help_line("#{":set o=val".fg("10")}", "Set option value")
     help_lines << help_line("#{"Options:".fg("245")}", "theme, wrap, fold_level")
     help_lines << help_line("#{"".fg("245")}", "show_numbers, tab_width")
-    help_lines << ""
-    
+    help_lines << " "
     help_lines << "#{"HELP & QUIT".fg("14")}"
     help_lines << help_line("#{"?".fg("10")}", "This help", "#{"??".fg("10")}", "Full documentation")
     help_lines << help_line("#{"q".fg("10")}", "Quit (asks to save)", "#{"Q".fg("10")}", "Force quit")
@@ -3838,13 +3838,17 @@ class HyperListApp
     saved_current = @current
     saved_offset = @offset
     saved_modified = @modified
-    
+    saved_presentation_mode = @presentation_mode
+
+    # Disable presentation mode for help viewer
+    @presentation_mode = false
+
     # Create temporary help items for scrolling
     @items = help.split("\n").map { |line| {"text" => line, "level" => 0, "fold" => false, "raw" => true} }
     @current = 0
     @offset = 0
     @modified = false
-    
+
     # Help viewer loop
     loop do
       render_main
@@ -3886,8 +3890,9 @@ class HyperListApp
     @current = saved_current
     @offset = saved_offset
     @modified = saved_modified
+    @presentation_mode = saved_presentation_mode
   end
-  
+
   def show_documentation
     doc = <<~DOC
     
@@ -4366,13 +4371,17 @@ class HyperListApp
     saved_offset = @offset
     saved_filename = @filename
     saved_modified = @modified
-    
+    saved_presentation_mode = @presentation_mode
+
+    # Disable presentation mode for documentation viewer
+    @presentation_mode = false
+
     # Create temporary documentation items
     @items = doc.split("\n").map { |line| {"text" => line, "level" => 0, "fold" => false, "raw" => true} }
     @current = 0
     @offset = 0
     @modified = false
-    
+
     # Documentation viewer loop
     loop do
       render_main
@@ -4419,8 +4428,9 @@ class HyperListApp
     @offset = saved_offset
     @filename = saved_filename
     @modified = saved_modified
+    @presentation_mode = saved_presentation_mode
   end
-  
+
   def handle_command
     @mode = :command
     # Set command history (reversed for proper UP arrow navigation)

data/hyperlist.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "hyperlist"
-  spec.version       = "1.9.1"
+  spec.version       = "1.9.3"
   spec.authors       = ["Geir Isene"]
   spec.email         = ["g@isene.com"]
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hyperlist
 version: !ruby/object:Gem::Version
-  version: 1.9.1
+  version: 1.9.3
 platform: ruby
 authors:
 - Geir Isene