hyperlist 1.5.0 → 1.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4e9aaa16290fdb66f194d55acfd1120e20b7b5c4f8b49203c20967580738cc4d
-  data.tar.gz: 8155928a2850fff84d471b287f45f38e490d907cb172dba0e18fbd8da06e18ea
+  metadata.gz: ec123fe286a3c475f76868ed8d0cdd1a512f7323c6da8d9446d2ea7e3913ce61
+  data.tar.gz: 637a405887c8725cd229ba0dbd8bfa0d0f75cffa787f336a59477a32a70b1669
 SHA512:
-  metadata.gz: 1df02a40d21b232190692f1bc7a2e08b516fa66e0a8a35743efd8743bceabe1548ea3b20bc3ab5a8f277558467af98c8bc8c8bece6322154471e40ba0dd2ab18
-  data.tar.gz: 8cbfa174bec6a8f3043942bd212f8b3b24548fe1ec1e42b4ef183f9db37bdd6dc0c934945bf99a29c85638b4a5d098c917d0e150c53feda7637e3e18f752075e
+  metadata.gz: c1ec3ebc57c928ab5c583da3ab2663cbaf73da09e36818f14e0973f3392583ebb5dc4ad1ee543f8263b386709677b70b73227b71c8145738f5afeee14e7a6af4
+  data.tar.gz: 8d2ff0119d45decfba5fbef08d96d19ebf92f63d0ce33fe35350a3a53cd92fbb94cc845fe85a951d746c374b87b2932c50f5bd54623b953edab162e8eee901c8

data/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 All notable changes to the HyperList Ruby TUI will be documented in this file.
 
+## [1.5.2] - 2025-09-18
+
+### Enhanced
+- **Smart movement behavior for collapsed items**
+  - LEFT/RIGHT arrow keys now move collapsed trees as a unit when item is folded
+  - Uncollapsed items move individually (only the current item)
+  - Maintains "move what's visible" principle for intuitive navigation
+  - Works in both normal and split-view modes
+
 ## [1.4.5] - 2025-09-02
 
 ### Added

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.4.3 - Terminal User Interface for HyperList files
+    HyperList v1.5.1 - 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.4.0"
+  puts "HyperList v1.5.1"
   exit 0
 end
 
@@ -72,7 +72,7 @@ class HyperListApp
   include Rcurses::Input
   include Rcurses::Cursor
   
-  VERSION = "1.5.0"
+  VERSION = "1.5.1"
   
   def initialize(filename = nil)
     @filename = filename ? File.expand_path(filename) : nil
@@ -135,9 +135,6 @@ class HyperListApp
       @cols = ENV['COLUMNS']&.to_i || 80
     end
     
-    # Debug: uncomment to see terminal size
-    # puts "Terminal size: #{@rows}x#{@cols}"
-    
     # Setup terminal resize handler
     setup_resize_handler
     
@@ -175,9 +172,8 @@ class HyperListApp
       # This is how RTFM works - it passes colored command output to panes with nil colors
       @main = Pane.new(1, 1, split_width - 1, @rows - 1, nil, nil)
       @split_pane = Pane.new(split_width + 1, 1, split_width - 1, @rows - 1, nil, nil)
-      # Footer: Use @rows for y-position (this puts it at the actual bottom)
       @footer = Pane.new(1, @rows, @cols, 1, 15, 8)
-      
+
       # Add separator with explicit background to overwrite emoji overflow
       @separator = Pane.new(split_width, 1, 1, @rows - 1, 15, 0)
       # Build separator with explicit clearing
@@ -192,7 +188,6 @@ class HyperListApp
       # Use nil colors like RTFM to avoid ANSI wrapping issues
       # This prevents the corruption that happens with narrow terminals
       @main = Pane.new(1, 1, @cols, @rows - 1, nil, nil)
-      # Footer: Use @rows for y-position (this puts it at the actual bottom)
       @footer = Pane.new(1, @rows, @cols, 1, 15, 8)
     end
   end
@@ -661,7 +656,6 @@ class HyperListApp
       content += "\n" unless content.end_with?("\n")
       content += "\n" + @config_line
     else
-      # Debug: Check why config line is missing
       if @fold_level != 99
         # Rebuild config line if we have config settings but no line
         update_config_line
@@ -1604,17 +1598,57 @@ class HyperListApp
   def has_children?(idx, items)
     return false if idx >= items.length - 1
     return false if idx < 0
-    
+
     current_item = items[idx]
     next_item = items[idx + 1]
-    
+
     # For visible items array
     if current_item.is_a?(Hash) && next_item.is_a?(Hash)
       return next_item["level"] > current_item["level"]
     end
-    
+
     false
   end
+
+  def should_move_with_children?
+    if @split_view && @active_pane == :split
+      # Handle split pane
+      visible_items = get_visible_split_items
+      return false if @split_current >= visible_items.length
+
+      item = visible_items[@split_current]
+      real_idx = @split_items.index(item)
+
+      # Move with children only if item is folded (collapsed) and has children
+      return real_idx && has_children_in_array?(real_idx, @split_items) && item["fold"]
+    else
+      # Handle main pane
+      visible = get_visible_items
+      return false if @current >= visible.length
+
+      item = visible[@current]
+      real_idx = get_real_index(item)
+
+      # Move with children only if item is folded (collapsed) and has children
+      return real_idx && has_children?(real_idx, @items) && @items[real_idx]["fold"]
+    end
+  end
+
+  def get_last_descendant_index(idx, items)
+    # Find the index after the last descendant of item at idx
+    # Returns idx + 1 if no children
+    return idx + 1 if idx >= items.length - 1
+    return idx + 1 unless has_children?(idx, items)
+
+    current_level = items[idx]["level"]
+    last_idx = idx + 1
+
+    while last_idx < items.length && items[last_idx]["level"] > current_level
+      last_idx += 1
+    end
+
+    last_idx
+  end
   
   def indent_split_right(with_children = false)
     visible_items = get_visible_split_items
@@ -2251,13 +2285,21 @@ class HyperListApp
   
   def insert_line_with_text(text)
     return unless text && !text.strip.empty?
-    
+
     save_undo_state  # Save state before modification
     visible = get_visible_items
     if @current < visible.length
       level = visible[@current]["level"]
       real_idx = get_real_index(visible[@current])
-      @items.insert(real_idx + 1, {"text" => text, "level" => level, "fold" => false})
+
+      # If current item has collapsed children, insert after all descendants
+      if @items[real_idx]["fold"] && has_children?(real_idx, @items)
+        insert_idx = get_last_descendant_index(real_idx, @items)
+      else
+        insert_idx = real_idx + 1
+      end
+
+      @items.insert(insert_idx, {"text" => text, "level" => level, "fold" => false})
     else
       @items << {"text" => text, "level" => 0, "fold" => false}
     end
@@ -2297,17 +2339,26 @@ class HyperListApp
   
   def insert_child_with_text(text)
     return unless text && !text.strip.empty?
-    
+
     save_undo_state  # Save state before modification
     visible = get_visible_items
     if @current < visible.length
       level = visible[@current]["level"] + 1
       real_idx = get_real_index(visible[@current])
-      
+
+      # If current item has collapsed children, insert as last child
+      if @items[real_idx]["fold"] && has_children?(real_idx, @items)
+        # Insert just before the next sibling (at the end of all descendants)
+        insert_idx = get_last_descendant_index(real_idx, @items)
+      else
+        # No collapsed children, insert right after current item
+        insert_idx = real_idx + 1
+      end
+
       # Unfold parent if needed
       @items[real_idx]["fold"] = false
-      
-      @items.insert(real_idx + 1, {"text" => text, "level" => level, "fold" => false})
+
+      @items.insert(insert_idx, {"text" => text, "level" => level, "fold" => false})
     else
       @items << {"text" => text, "level" => 1, "fold" => false}
     end
@@ -4003,13 +4054,6 @@ class HyperListApp
       begin
         c = getchr
         
-        # Debug logging
-        if ENV['DEBUG']
-          File.open('/tmp/hyperlist_debug.log', 'a') do |f|
-            f.puts "Password prompt key: #{c.inspect}"
-          end
-        end
-        
         case c
         when "ENTER", "\r", "\n"
           break
@@ -4676,11 +4720,6 @@ class HyperListApp
         # Skip truly nil input
         next if c.nil?
         
-        # Debug logging to file
-        File.open('/tmp/hyperlist_debug.log', 'a') do |f|
-          f.puts "Template key received: #{c.inspect} (length: #{c.length}, ord: #{c.bytes.inspect})"
-        end if ENV['DEBUG']
-        
         case c
         when "q", "ESC", "C-c", "Q"
           exit_loop = true
@@ -4868,16 +4907,20 @@ class HyperListApp
         indent_left(true)  # with children
       end
     when "RIGHT"
+      # Move collapsed trees as a unit, but only the item if uncollapsed
+      should_move_children = should_move_with_children?
       if @split_view && @active_pane == :split
-        indent_split_right(false)
+        indent_split_right(should_move_children)
       else
-        indent_right(false)
+        indent_right(should_move_children)
       end
     when "LEFT"
+      # Move collapsed trees as a unit, but only the item if uncollapsed
+      should_move_children = should_move_with_children?
       if @split_view && @active_pane == :split
-        indent_split_left(false)
+        indent_split_left(should_move_children)
       else
-        indent_left(false)
+        indent_left(should_move_children)
       end
     when " "
       toggle_fold
@@ -5666,18 +5709,20 @@ class HyperListApp
         when "l"
           go_to_first_child
         when "LEFT"
-          # Unindent only the current item
+          # Move collapsed trees as a unit, but only the item if uncollapsed
+          should_move_children = should_move_with_children?
           if @split_view && @active_pane == :split
-            indent_split_left(false)
+            indent_split_left(should_move_children)
           else
-            indent_left(false)
+            indent_left(should_move_children)
           end
         when "RIGHT"
-          # Indent only the current item
+          # Move collapsed trees as a unit, but only the item if uncollapsed
+          should_move_children = should_move_with_children?
           if @split_view && @active_pane == :split
-            indent_split_right(false)
+            indent_split_right(should_move_children)
           else
-            indent_right(false)
+            indent_right(should_move_children)
           end
         when "PgUP"  # Page Up
           page_up

data/hyperlist.gemspec

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hyperlist
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 1.5.2
 platform: ruby
 authors:
 - Geir Isene
 autorequire:
 bindir: "."
 cert_chain: []
-date: 2025-09-03 00:00:00.000000000 Z
+date: 2025-09-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rcurses