From 865a7ecc7f4cedee12963cecf5a9a182be81f3e5 Mon Sep 17 00:00:00 2001
From: leekelleher <leekelleher@gmail.com>
Date: Mon, 19 Feb 2024 13:46:25 +0000
Subject: [PATCH] `umb-table` fixes issue with `icon`

The `item.icon` is nullable, which isn't picked up
by the `ifDefined` check, so causes a Typescript error.

Also amends CSS to align the icon with the swappable checkbox.
---
 .../src/packages/core/components/table/table.element.ts     | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/Umbraco.Web.UI.Client/src/packages/core/components/table/table.element.ts b/src/Umbraco.Web.UI.Client/src/packages/core/components/table/table.element.ts
index a10cd863ba..617f4a0343 100644
--- a/src/Umbraco.Web.UI.Client/src/packages/core/components/table/table.element.ts
+++ b/src/Umbraco.Web.UI.Client/src/packages/core/components/table/table.element.ts
@@ -219,7 +219,7 @@ export class UmbTableElement extends LitElement {
 		if (this.config.hideIcon && !this.config.allowSelection) return;
 
 		return html`<uui-table-cell>
-			${when(!this.config.hideIcon, () => html`<uui-icon name=${ifDefined(item.icon)}></uui-icon>`)}
+			${when(!this.config.hideIcon, () => html`<uui-icon name=${ifDefined(item.icon ?? undefined)}></uui-icon>`)}
 			${when(
 				this.config.allowSelection,
 				() =>
@@ -311,6 +311,10 @@ export class UmbTableElement extends LitElement {
 				justify-content: space-between;
 				width: 100%;
 			}
+
+			uui-table-cell uui-icon {
+				vertical-align: top;
+			}
 		`,
 	];
 }