Muti line text support by BrunoVT1992 · Pull Request #21 · BrunoVT1992/ConsoleTable

BrunoVT1992 · 2026-04-05T12:50:35Z

No description provided.

Copilot

Pull request overview

Adds multi-line cell rendering support to ConsoleTable.Text so headers, rows, and footers can contain newline characters and render as multiple physical lines with correct column sizing.

Changes:

Updated table rendering logic to split cell content by line breaks and render multi-line rows/headers/footers.
Added an extensive test suite covering multi-line rendering, alignment, padding, CRLF/LF handling, and mixed row types.
Updated docs/examples and bumped package version + changelog entry for v2.2.0.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
`ConsoleTable.Text/Table.cs`	Implements multi-line splitting, width calculation based on widest line, and multi-line rendering in `CreateValueLine`.
`Tests/ConsoleTable.Text.Tests/TableTests.cs`	Adds multi-line behavior tests and helper for output line parsing.
`README.md`	Documents multi-line support and updates Quick Start example/output.
`ConsoleTable.Text.Examples/Program.cs`	Adds multi-line example and updates some sample rows to include newlines.
`ConsoleTable.Text/ConsoleTable.Text.csproj`	Bumps package version to 2.2.0.
`ConsoleTable.slnx`	Includes the new 2.2.0 changelog file in the solution items.
`ChangeLogs/2.2.0-ChangeLog.md`	Adds v2.2.0 release notes with a multi-line usage example and output.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-05T12:56:29Z

ConsoleTable.Text/Table.cs

            foreach (var row in table)
            {
                if (row == null || row.Length <= 0)
                    continue;



ToTable() calls CleanupRow (which converts null/empty rows into a single-column row), but GetMaxCellWidths() skips null/empty rows entirely. If a table contains only an empty row (e.g., AddRow() with no values) and no headers/footers, maximumCellWidths can end up length 0 and CreateValueLine() will throw IndexOutOfRangeException when rendering the cleaned row. Consider applying CleanupRow inside GetMaxCellWidths() (and when determining the initial column count) so widths are computed for the same normalized rows that are rendered.

Test validates empty table does not throw exception

Tests/ConsoleTable.Text.Tests/TableTests.cs

Copilot · 2026-04-05T12:56:29Z

README.md

 ```csharp
 using ConsoleTable.Text;



The Quick Start snippet uses Environment.NewLine and List<string[]> later in the example, but only imports ConsoleTable.Text. To make the snippet compile in projects without implicit global usings, add using System; and using System.Collections.Generic; (or fully-qualify those types).

Only an exanple

README.md

Copilot · 2026-04-05T12:56:30Z

ConsoleTable.Text.Examples/Program.cs

+                new string[] { "Alice Cooper", $"30{Environment.NewLine}1995", "New York" },
+                new string[] { "Bob", $"25{Environment.NewLine}2000", "Los Angeles" },
+                new string[] { "Charlie Brown", $"67{Environment.NewLine}1958", "Chicago" },
+                new string[] { "Gloria", $"40{Environment.NewLine}1985", $"Chicago{Environment.NewLine}Originally fron Bogota, Colombia" }


Typo in the example string: "Originally fron Bogota, Colombia" should be "Originally from Bogota, Colombia".

Suggested change

new string[] { "Gloria", $"40{Environment.NewLine}1985", $"Chicago{Environment.NewLine}Originally fron Bogota, Colombia" }

new string[] { "Gloria", $"40{Environment.NewLine}1985", $"Chicago{Environment.NewLine}Originally from Bogota, Colombia" }

Copilot · 2026-04-05T12:56:30Z

ChangeLogs/2.2.0-ChangeLog.md

+Cell content can contain newlines and the table will adjust row height accordingly, in headers, rows and footers.
+
+
+```csharp


The changelog snippet uses Environment.NewLine and List<string[]>, but only imports ConsoleTable.Text. Consider adding using System; and using System.Collections.Generic; so the example compiles without relying on implicit global usings.

Suggested change

```csharp

```csharp

using System;

using System.Collections.Generic;

Copilot · 2026-04-05T12:56:30Z

ChangeLogs/2.2.0-ChangeLog.md

+        new string[] { "Charlie Brown", $"67{Environment.NewLine}1958", "Chicago" },
+        new string[] { "Gloria", $"40{Environment.NewLine}1985", $"Chicago{Environment.NewLine}Originally fron Bogota, Colombia" }
+    },


Typo in the example string: "Originally fron Bogota, Colombia" should be "Originally from Bogota, Colombia".

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…2/ConsoleTable into feature/multiLineText

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-05T13:48:16Z

ConsoleTable.Text/Table.cs

            foreach (var row in table)
            {
                if (row == null || row.Length <= 0)
                    continue;

                for (int i = 0; i < row.Length; i++)
                {
-                    var maxWidth = row[i].Length + paddingCount;
+                    var lines = GetCellLines(row[i]);


GetMaxCellWidths() skips rows that are null or have Length <= 0, but ToTable() later calls CleanupRow() which converts those rows into a 1-column row (new[] { " " }). If Rows contains only an empty/null row (e.g., AddRow() with no values or AddRow(null)), maximumCellWidths can end up length 0 and CreateValueLine() will throw IndexOutOfRangeException when it tries to access maximumCellWidths[0]. Normalize rows consistently when computing widths (e.g., apply CleanupRow() while building/iterating table, and ensure maximumColumns is at least 1 when any row exists after cleanup).

Copilot · 2026-04-05T13:48:16Z

Tests/ConsoleTable.Text.Tests/TableTests.cs

+    [Fact]
+    public void Empty()
+    {
+        var table = new Table();
+
+        var result = table.ToTable();
+
+        Assert.Equal(string.Empty, result);
+    }


Test method name Empty doesn’t follow the naming pattern used throughout this file (e.g., SetHeaders_Success, ToTable_...). Consider renaming it to something descriptive like ToTable_EmptyTable_ReturnsEmptyString to keep the suite consistent and self-explanatory.

Copilot · 2026-04-05T13:48:16Z

README.md

 - Simple and intuitive API
 - Optimized for performance
 - Support for varying column counts across rows (each row can have its own number of cells)
+- Multi line support (cell content can contain newlines and the table will adjust row height accordingly) in headers, rows and footers


Typo/wording: consider using the hyphenated form “Multi-line support” (and “multi-line” as an adjective) for standard English usage.

Suggested change

- Multi line support (cell content can contain newlines and the table will adjust row height accordingly) in headers, rows and footers

- Multi-line support (cell content can contain newlines and the table will adjust row height accordingly) in headers, rows and footers

Copilot · 2026-04-05T13:48:17Z

ChangeLogs/2.2.0-ChangeLog.md

@@ -0,0 +1,53 @@
+# V2.2.0
+
+## Multi line support


Typo/wording: use “Multi-line support” (hyphenated) in the section header for standard English usage (and match README phrasing).

Suggested change

## Multi line support

## Multi-line support

Copilot · 2026-04-05T13:48:17Z

ConsoleTable.Text.Examples/Program.cs

+    private static void WriteMultiLineTable()
+    {
+        Console.WriteLine();
+        Console.WriteLine("Multi line table:");


Minor wording: consider changing the output label to “Multi-line table:” for consistency with the README/changelog phrasing.

Suggested change

Console.WriteLine("Multi line table:");

Console.WriteLine("Multi-line table:");

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ConsoleTable.Text/Table.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Bruno Van Thournout added 6 commits April 5, 2026 14:11

Add multi line support

8aee213

Update default table

d075c7a

Update big table to multi row support

9821611

Update version

32cc584

Update cgangelog

c26a642

Fix code

6196922

BrunoVT1992 requested a review from Copilot April 5, 2026 12:52

Copilot started reviewing on behalf of BrunoVT1992 April 5, 2026 12:52 View session

Copilot AI reviewed Apr 5, 2026

View reviewed changes

Bruno Van Thournout and others added 6 commits April 5, 2026 15:39

Test empty table

2b68eac

Update Tests/ConsoleTable.Text.Tests/TableTests.cs

b557561

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update README.md

d801f5a

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Fix readme

52dd737

Merge branch 'feature/multiLineText' of https://github.com/BrunoVT199…

cbf3413

…2/ConsoleTable into feature/multiLineText

Fix typo

e72257d

BrunoVT1992 requested a review from Copilot April 5, 2026 13:44

Copilot started reviewing on behalf of BrunoVT1992 April 5, 2026 13:45 View session

Copilot AI reviewed Apr 5, 2026

View reviewed changes

Bruno Van Thournout added 4 commits April 6, 2026 15:28

Implement more tests

56a1fc9

Add more temp example

3765a59

Text drawing fixes

935a9c2

Add release notes

34e06b9

BrunoVT1992 requested a review from Copilot April 6, 2026 13:43

Fix typos

2edbaa0

Copilot AI reviewed Apr 6, 2026

View reviewed changes

ConsoleTable.Text/Table.cs Outdated Show resolved Hide resolved

Update ConsoleTable.Text/Table.cs

d898830

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

BrunoVT1992 merged commit 6357abb into master Apr 6, 2026
2 checks passed

BrunoVT1992 deleted the feature/multiLineText branch April 6, 2026 13:51

	new string[] { "Gloria", $"40{Environment.NewLine}1985", $"Chicago{Environment.NewLine}Originally fron Bogota, Colombia" }
	new string[] { "Gloria", $"40{Environment.NewLine}1985", $"Chicago{Environment.NewLine}Originally from Bogota, Colombia" }

		Cell content can contain newlines and the table will adjust row height accordingly, in headers, rows and footers.


		```csharp

	- Multi line support (cell content can contain newlines and the table will adjust row height accordingly) in headers, rows and footers
	- Multi-line support (cell content can contain newlines and the table will adjust row height accordingly) in headers, rows and footers

	Console.WriteLine("Multi line table:");
	Console.WriteLine("Multi-line table:");

Conversation

BrunoVT1992 commented Apr 5, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

BrunoVT1992 Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

BrunoVT1992 Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants