From bca1c889d140d43d9041e89226f127517e005275 Mon Sep 17 00:00:00 2001
From: Martin Andersen <martin.andersen@outlook.com>
Date: Mon, 26 Oct 2020 20:41:49 +0100
Subject: [PATCH] Add Padding extension

Add extension method for specifying horizontal and vertical padding.
Similar constructor for `Padding` already existed.

Update documentation for table column appearance (padding).

Update
`Should_Render_Padded_Object_Correctly_When_Nested_Within_Other_Object`
test to use new extension method.
---
 docs/input/widgets/table.md                      | 10 +++++++---
 src/Spectre.Console.Tests/Unit/PadderTests.cs    |  2 +-
 .../Extensions/PaddableExtensions.cs             | 16 +++++++++++++++-
 3 files changed, 23 insertions(+), 5 deletions(-)

diff --git a/docs/input/widgets/table.md b/docs/input/widgets/table.md
index 7db0545e..d124287e 100644
--- a/docs/input/widgets/table.md
+++ b/docs/input/widgets/table.md
@@ -109,12 +109,16 @@ table.Columns[0].RightAligned();
 ## Padding
 
 ```csharp
-// Set left and right padding
-table.Columns[0].Padding(left: 3, right: 5);
-
 // Set padding individually
 table.Columns[0].PadLeft(3);
 table.Columns[0].PadRight(5);
+
+// Or chained together
+table.Columns[0].PadLeft(3).PadRight(5);
+
+// Or with the shorthand method if the left and right 
+// padding are identical. Vertical padding is ignored.
+table.Columns[0].Padding(4, 0);
 ```
 
 ## Disable column wrapping
diff --git a/src/Spectre.Console.Tests/Unit/PadderTests.cs b/src/Spectre.Console.Tests/Unit/PadderTests.cs
index 5f8cb060..b413dcda 100644
--- a/src/Spectre.Console.Tests/Unit/PadderTests.cs
+++ b/src/Spectre.Console.Tests/Unit/PadderTests.cs
@@ -77,7 +77,7 @@ namespace Spectre.Console.Tests.Unit
             table.AddColumn("Bar", c => c.PadLeft(0).PadRight(0));
             table.AddRow("Baz", "Qux");
             table.AddRow(new Text("Corgi"), new Padder(new Panel("Waldo"))
-                .Padding(2, 1, 2, 1));
+                .Padding(2, 1));
 
             // When
             console.Render(new Padder(table)
diff --git a/src/Spectre.Console/Extensions/PaddableExtensions.cs b/src/Spectre.Console/Extensions/PaddableExtensions.cs
index ad1ea739..ca038470 100644
--- a/src/Spectre.Console/Extensions/PaddableExtensions.cs
+++ b/src/Spectre.Console/Extensions/PaddableExtensions.cs
@@ -80,7 +80,7 @@ namespace Spectre.Console
         }
 
         /// <summary>
-        /// Sets the left and right padding.
+        /// Sets the left, top, right and bottom padding.
         /// </summary>
         /// <typeparam name="T">An object implementing <see cref="IPaddable"/>.</typeparam>
         /// <param name="obj">The paddable object instance.</param>
@@ -95,6 +95,20 @@ namespace Spectre.Console
             return Padding(obj, new Padding(left, top, right, bottom));
         }
 
+        /// <summary>
+        /// Sets the horizontal and vertical padding.
+        /// </summary>
+        /// <typeparam name="T">An object implementing <see cref="IPaddable"/>.</typeparam>
+        /// <param name="obj">The paddable object instance.</param>
+        /// <param name="horizontal">The left and right padding.</param>
+        /// <param name="vertical">The top and bottom padding.</param>
+        /// <returns>The same instance so that multiple calls can be chained.</returns>
+        public static T Padding<T>(this T obj, int horizontal, int vertical)
+            where T : class, IPaddable
+        {
+            return Padding(obj, new Padding(horizontal, vertical));
+        }
+
         /// <summary>
         /// Sets the padding.
         /// </summary>