Skip to content

table: stack vertically-merged rows into wrapped cell height; fixes #261 - #411

Merged
jedib0t merged 1 commit into
mainfrom
fix-261-automerge-wrap
Jul 19, 2026
Merged

table: stack vertically-merged rows into wrapped cell height; fixes #261#411
jedib0t merged 1 commit into
mainfrom
fix-261-automerge-wrap

Conversation

@jedib0t

@jedib0t jedib0t commented Jul 19, 2026

Copy link
Copy Markdown
Owner

When a column has both AutoMerge and WidthMax, a merged cell rendered blank but still forced the row to the pre-merge wrapped height, leaving trailing blank lines below the merged content.

Body rows spanned by the same merged cell are now rendered as one shared block so the differing columns slot into the merged cell's wrapped height instead of leaving blanks below it. The block is the full span of a merged cell, and any inner merged column does its own vertical merge within it. This is gated to the plain case (no row separators, auto-index, pager, or per-row coloring) and only kicks in when the stacked rows each fit on a single line, so it never changes any other layout; everything else falls back to trimming the trailing blank lines.

@coveralls

coveralls commented Jul 19, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 29704631714

Coverage remained the same at 100.0%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 131 of 131 lines across 2 files are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 5155
Covered Lines: 5155
Line Coverage: 100.0%
Coverage Strength: 1.21 hits per line

💛 - Coveralls

@jedib0t
jedib0t force-pushed the fix-261-automerge-wrap branch from 0fddb63 to b49decf Compare July 19, 2026 21:35
@sonarqubecloud

Copy link
Copy Markdown

@jedib0t
jedib0t merged commit 7f91001 into main Jul 19, 2026
4 checks passed
@jedib0t
jedib0t deleted the fix-261-automerge-wrap branch July 19, 2026 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants