try-cli 1.10.0 → 1.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2886940b1f4b531eac99e082e57083e26e0aedc60f073355868870fc364118c7
-  data.tar.gz: 64712394de4b5d48535ce8dc31abb2f50da33e446aed3d01ec465640191c6b0e
+  metadata.gz: 6bc9090ddafc7354bd7d4207e5247e9afd3f52df65e43c1600e21f717c3644cb
+  data.tar.gz: cc3fbc4c3868d6d504e326702329fa14d377257d9300c468c07f929f5137c176
 SHA512:
-  metadata.gz: a13726660cb156fde7e39a351b439d3a64870fcc637a4fe4189c54a0b3a2ad417f3685ec9a010ed65b7b1f254ee26b46e6b20f1c25ff8ca1caf5ae0d1b1852f1
-  data.tar.gz: 448bb7c503410c82052e93ce8dc6b22922be70c80dd1a8c0a9f8c7e84e7f50bc0efa4d435c8c8bec4c0578cb8969185cc624fcd3169e9421eead312aa84f4223
+  metadata.gz: 21c62da72659e06ca9614d1ee82f8063f80f13b153e53af39102cdb21e083ab00c46d0d97c42a91cbbe8e0142d5030448660fd9594790bb4da753a0f8314de20
+  data.tar.gz: 83376a51f4be2be9fe9dcb4c0de857174fd53d1409ec65b1e1f6cc5d9728a4bcff5bfa800fe180e63f799aab1063c207ab8a6ab11196351715454a3f934819ab

data/VERSION

@@ -1 +1 @@
-1.10.0
+1.10.1

data/lib/tui.rb

@@ -90,14 +90,15 @@ module Tui
   module Palette
     HEADER      = ANSI.sgr(1, "38;5;114")
     ACCENT      = ANSI.sgr(1, "38;5;214")
-    HIGHLIGHT   = "\e[1m"  # Bold only, no fixed color: readable on any theme
+    HIGHLIGHT   = "\e[1;33m"  # Bold yellow remains readable on the controlled selected background
     MUTED       = ANSI.fg(245)
     MATCH       = ANSI.sgr(1, "38;5;226")
     INPUT_HINT  = ANSI.fg(244)
     INPUT_CURSOR_ON  = "\e[7m"
     INPUT_CURSOR_OFF = "\e[27m"
 
-    SELECTED_BG = "\e[7m"  # reverse video: adapts to terminal's own colors, not just dark themes
+    SELECTED_BG = ANSI.bg(238) # Preserve the dark selected-row appearance
+    SELECTED_FG = ANSI.fg(255) # Keep selected content readable on every theme
     DANGER_BG   = ANSI.bg(52)
   end
 

data/try.rb

@@ -389,15 +389,15 @@ class TrySelector
 
   def render_entry_line(screen, entry, is_selected, width)
     is_marked = @marked_for_deletion.include?(entry[:path])
-    # Marked items always show red; selection shows via arrow only
+    # Marked items keep the danger background; selected rows add a readable foreground.
     background = if is_marked
-      Tui::Palette::DANGER_BG
+      Tui::Palette::DANGER_BG + (is_selected ? Tui::Palette::SELECTED_FG : "")
     elsif is_selected
-      Tui::Palette::SELECTED_BG
+      Tui::Palette::SELECTED_BG + Tui::Palette::SELECTED_FG
     end
 
     line = screen.body.add_line(background: background)
-    line.write << (is_selected ? Tui::Text.highlight("→ ") : "  ")
+    line.write << (is_selected ? Tui::Text.highlight("→ ") + selected_foreground : "  ")
     icon = if is_marked
       emoji("🗑️")
     elsif entry[:is_symlink]
@@ -407,7 +407,7 @@ class TrySelector
     end
     line.write << icon << " "
 
-    plain_name, rendered_name = formatted_entry_name(entry)
+    plain_name, rendered_name = formatted_entry_name(entry, selected: is_selected)
     prefix_width = 5
     meta_text = "#{format_relative_time(entry[:mtime])}, #{format('%.1f', entry[:score])}"
 
@@ -422,13 +422,15 @@ class TrySelector
     line.write << display_rendered
 
     # Right content is lower layer - will be overwritten by left if they overlap
-    line.right.write_dim(meta_text)
+    line.right.write(is_selected ? meta_text : Tui::Text.dim(meta_text))
   end
 
   def render_create_line(screen, is_selected, width)
-    background = is_selected ? Tui::Palette::SELECTED_BG : nil
+    background = if is_selected
+      Tui::Palette::SELECTED_BG + Tui::Palette::SELECTED_FG
+    end
     line = screen.body.add_line(background: background)
-    line.write << (is_selected ? Tui::Text.highlight("→ ") : "  ")
+    line.write << (is_selected ? Tui::Text.highlight("→ ") + selected_foreground : "  ")
     date_prefix = Time.now.strftime("%Y-%m-%d")
     label = if @input_buffer.empty?
       "📂 Create new: #{date_prefix}-"
@@ -438,7 +440,7 @@ class TrySelector
     line.write << label
   end
 
-  def formatted_entry_name(entry)
+  def formatted_entry_name(entry, selected: false)
     basename = entry[:basename]
     positions = entry[:highlight_positions] || []
 
@@ -447,17 +449,25 @@ class TrySelector
       name_part = $2
       date_len = date_part.length + 1  # +1 for the hyphen
 
-      rendered = Tui::Text.dim(date_part)
+      rendered = selected ? date_part : Tui::Text.dim(date_part)
       # Highlight hyphen if it's in positions
-      rendered += positions.include?(10) ? Tui::Text.highlight('-') : Tui::Text.dim('-')
-      rendered += highlight_with_positions(name_part, positions, date_len)
+      hyphen = if positions.include?(10)
+        Tui::Text.highlight('-')
+      elsif selected
+        '-'
+      else
+        Tui::Text.dim('-')
+      end
+      rendered += hyphen
+      rendered += selected_foreground if selected && positions.include?(10)
+      rendered += highlight_with_positions(name_part, positions, date_len, selected: selected)
       ["#{date_part}-#{name_part}", rendered]
     else
-      [basename, highlight_with_positions(basename, positions, 0)]
+      [basename, highlight_with_positions(basename, positions, 0, selected: selected)]
     end
   end
 
-  def highlight_with_positions(text, positions, offset)
+  def highlight_with_positions(text, positions, offset, selected: false)
     pos_set = positions.is_a?(Set) ? positions : positions.to_set
     result = String.new
     chars = text.chars
@@ -469,6 +479,7 @@ class TrySelector
         i += 1
         i += 1 while i < chars.length && pos_set.include?(i + offset)
         result << Tui::Text.highlight(chars[batch_start...i].join)
+        result << selected_foreground if selected
       else
         result << chars[i]
         i += 1
@@ -477,6 +488,10 @@ class TrySelector
     result
   end
 
+  def selected_foreground
+    Tui.colors_enabled? ? Tui::Palette::SELECTED_FG : ""
+  end
+
   # Find the position of the previous word boundary for Ctrl-W deletion.
   # Skips non-alphanumeric chars, then skips alphanumeric chars.
   def word_boundary_backward(buffer, cursor)
@@ -955,7 +970,7 @@ end
 # Main execution with OptionParser subcommands
 if __FILE__ == $0
 
-  VERSION = "1.10.0"
+  VERSION = "1.10.1"
 
   def print_global_help
     text = <<~HELP

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: try-cli
 version: !ruby/object:Gem::Version
-  version: 1.10.0
+  version: 1.10.1
 platform: ruby
 authors:
 - Tobi Lutke