Skip to content

feat/ Add cell wrapping to fit column width#52

Open
kibi2 wants to merge 4 commits intomainfrom
refactor/line-fetch-with-cache
Open

feat/ Add cell wrapping to fit column width#52
kibi2 wants to merge 4 commits intomainfrom
refactor/line-fetch-with-cache

Conversation

@kibi2
Copy link
Copy Markdown
Owner

@kibi2 kibi2 commented Apr 8, 2026

Summary

This PR introduces cell content wrapping based on column width.

When a cell's content exceeds the available column width, it is now split into multiple lines so that it fits within the column constraints. This enables proper multi-line rendering of table cells.

Details

  • Implemented wrapping logic for cell content
  • Ensures each line does not exceed the specified column width
  • Supports multi-byte characters correctly

Motivation

Previously, long cell content could overflow or break layout assumptions. With this change, table rendering becomes more robust and visually consistent, especially when column widths are constrained.

Notes

  • This change mainly affects rendering behavior
  • No breaking changes expected in external interfaces

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 8, 2026

Codecov Report

❌ Patch coverage is 97.23320% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lua/tirenvi/core/blocks.lua 92.00% 2 Missing ⚠️
lua/tirenvi/core/cell.lua 94.44% 2 Missing ⚠️
lua/tirenvi/core/attr.lua 90.90% 1 Missing ⚠️
lua/tirenvi/core/block.lua 98.30% 1 Missing ⚠️
lua/tirenvi/core/repair.lua 95.23% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@kibi2 kibi2 force-pushed the refactor/line-fetch-with-cache branch 2 times, most recently from 1c6b9e5 to df10622 Compare April 8, 2026 02:31
@kibi2 kibi2 force-pushed the refactor/line-fetch-with-cache branch from df10622 to bb52048 Compare April 8, 2026 07:54
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.

1 participant