marvi 0.4.1 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 519a600dd8ba2e4c6934cf57f37914337ce252ced2a51d6b05dff2ab2c1dda4b
-  data.tar.gz: '02669ac614cde421459555a5068429e1371036dd90eb549f5a90d6f3e3aea90c'
+  metadata.gz: d5ff0488e214dd427a028dabe9d240bee43e48ac27cef895e7f8584b41695435
+  data.tar.gz: 71dcd3de576d8926b90cae6ec4b79732be3627b85922351585b624b064eafb94
 SHA512:
-  metadata.gz: 37a67589e93f21f02d2c7cec09130e2a63ce83e88ebe21c61cf643b4810506ff4ece12ca39d8c36a40cbccaca35ecf8984e1c8276f953b0e17bf2b9caf4f98de
-  data.tar.gz: 12fa9234e6fb3dd1b13b72b5008a239acea242b049ae4086d6e2a8565435b4727fd5850c5766d79f05bbc48332719f7a2b97609725c8834ca1f6083d4a7acf02
+  metadata.gz: 1f90d2999b6336d1fa26f8ef441c8e51e551f4f7d229633053acc9ab8418fe115bcd6ea3f8fa3b67ce678781b29198acc33fd1ed69f591611a47bc97b456901c
+  data.tar.gz: 82bcdfbe5166bcd2f48d2367da468df0290c89e831d16fa06dced445dba98a2ba0846c659a87e9f69a1dd2e68c900f9cccdaaa57790bb0f35ddcb2c4b8d8dc0e

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 ## [Unreleased]
 
+## [0.4.2] - 2026-05-31
+
+- Add left/right padding in the curses pager so content no longer sits flush against the terminal edges. Padding scales with terminal width.
+- Render every line of a multi-line blockquote with its `│ ` prefix.
+- Stop emitting an empty `│ ` line at the end of blockquotes.
+- Collapse consecutive blank lines so block-level elements are separated by a single blank line.
+
 ## [0.4.1] - 2026-05-26
 
 - Wrap long text in bulleted/ordered lists, paragraphs, headers, and blockquotes so it no longer overflows the terminal width. List items and headers use a hanging indent for continuation lines. (#1)

data/lib/marvi/ast_walker.rb

@@ -16,7 +16,7 @@ module Marvi
       doc = Kramdown::Document.new(markdown, input: "GFM")
       lines = render_block(doc.root)
       lines.pop while lines.last&.plain_text&.empty?
-      lines
+      collapse_consecutive_blanks(lines)
     end
 
     private
@@ -131,9 +131,26 @@ module Marvi
       @max_width = [saved_width - prefix_width, MIN_COL_WIDTH].max
       inner = el.children.flat_map { |child| render_block(child) }
       @max_width = saved_width
+      # Trim trailing blanks and collapse runs of blanks so the │ prefix doesn't produce
+      # dangling/duplicated "│ " lines (kramdown emits both :p trailing blanks and explicit
+      # :blank elements between siblings).
+      inner.pop while inner.last&.plain_text&.empty?
+      inner = collapse_consecutive_blanks(inner)
       inner.map { |line| RichLine.new([prefix] + line.spans, source_line: line.source_line) } + [RichLine.blank]
     end
 
+    def collapse_consecutive_blanks(lines)
+      out = []
+      prev_blank = false
+      lines.each do |line|
+        is_blank = line.plain_text.empty?
+        next if is_blank && prev_blank
+        out << line
+        prev_blank = is_blank
+      end
+      out
+    end
+
     def render_table(el)
       src = el.options[:location]
       rows = el.children.flat_map(&:children)
@@ -210,34 +227,43 @@ module Marvi
       lines = [[]]
       current_width = 0
       spans.each do |span|
-        text = span.text.dup
-        until text.empty?
-          remaining = width - current_width
-          if remaining <= 0
+        # Split on \n so embedded hard line breaks become separate RichLines downstream,
+        # otherwise Curses.addstr resets the cursor to column 0 and bypasses padding/prefix.
+        parts = span.text.split("\n", -1)
+        parts.each_with_index do |part, idx|
+          if idx > 0 && !lines.last.empty?
             lines << []
             current_width = 0
-            remaining = width
-          end
-          taken = 0
-          chunk_width = 0
-          text.each_char do |c|
-            cw = Unicode::DisplayWidth.of(c)
-            break if chunk_width + cw > remaining
-            chunk_width += cw
-            taken += c.bytesize
-          end
-          if taken.zero?
-            first_char = text.each_char.first
-            taken = first_char.bytesize
-            chunk_width = Unicode::DisplayWidth.of(first_char)
           end
-          chunk = text.byteslice(0, taken)
-          text = text.byteslice(taken..) || ""
-          lines.last << Span.new(text: chunk, bold: span.bold, italic: span.italic, color: span.color, bg_color: span.bg_color)
-          current_width += chunk_width
-          unless text.empty?
-            lines << []
-            current_width = 0
+          text = part.dup
+          until text.empty?
+            remaining = width - current_width
+            if remaining <= 0
+              lines << []
+              current_width = 0
+              remaining = width
+            end
+            taken = 0
+            chunk_width = 0
+            text.each_char do |c|
+              cw = Unicode::DisplayWidth.of(c)
+              break if chunk_width + cw > remaining
+              chunk_width += cw
+              taken += c.bytesize
+            end
+            if taken.zero?
+              first_char = text.each_char.first
+              taken = first_char.bytesize
+              chunk_width = Unicode::DisplayWidth.of(first_char)
+            end
+            chunk = text.byteslice(0, taken)
+            text = text.byteslice(taken..) || ""
+            lines.last << Span.new(text: chunk, bold: span.bold, italic: span.italic, color: span.color, bg_color: span.bg_color)
+            current_width += chunk_width
+            unless text.empty?
+              lines << []
+              current_width = 0
+            end
           end
         end
       end

data/lib/marvi/renderer/curses.rb

@@ -23,6 +23,9 @@ module Marvi
       CTRL_D = 4
       CTRL_U = 21
 
+      MIN_HORIZONTAL_PADDING = 2
+      HORIZONTAL_PADDING_DIVISOR = 12
+
       def render(markdown, file: nil)
         @file = file
         @markdown = markdown
@@ -127,7 +130,17 @@ module Marvi
       end
 
       def rewalk
-        @lines = ASTWalker.new.walk(@markdown, max_width: ::Curses.cols)
+        @lines = ASTWalker.new.walk(@markdown, max_width: content_width)
+      end
+
+      def horizontal_padding
+        cols = ::Curses.cols
+        return 0 if cols <= MIN_HORIZONTAL_PADDING * 2
+        [MIN_HORIZONTAL_PADDING, cols / HORIZONTAL_PADDING_DIVISOR].max
+      end
+
+      def content_width
+        [::Curses.cols - horizontal_padding * 2, 1].max
       end
 
       def reload_from_key
@@ -214,8 +227,9 @@ module Marvi
 
       def draw
         ::Curses.clear
+        padding = horizontal_padding
         visible_lines.each_with_index do |line, row|
-          ::Curses.setpos(row, 0)
+          ::Curses.setpos(row, padding)
           render_line(line)
         end
         draw_status_bar

data/lib/marvi/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Marvi
-  VERSION = "0.4.1"
+  VERSION = "0.4.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: marvi
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - Mitsutaka Mimura