From c16a31c3da1b6bcde11e5d01f2a842a443e048b3 Mon Sep 17 00:00:00 2001
From: akdalin <dalin.akrasi@nhs.net>
Date: Tue, 29 Apr 2025 15:49:29 +0100
Subject: [PATCH 1/2] Create Hero View, ViewComponent and Model

---
 .../ViewComponents/HeroViewComponent.cs       | 18 ++++++++++
 .../ViewModels/HeroViewModel.cs               | 23 +++++++++++++
 .../Shared/Components/Hero/Default.cshtml     | 33 +++++++++++++++++++
 3 files changed, 74 insertions(+)
 create mode 100644 DotnetViewComponents/ViewComponents/HeroViewComponent.cs
 create mode 100644 DotnetViewComponents/ViewModels/HeroViewModel.cs
 create mode 100644 DotnetViewComponents/Views/Shared/Components/Hero/Default.cshtml

diff --git a/DotnetViewComponents/ViewComponents/HeroViewComponent.cs b/DotnetViewComponents/ViewComponents/HeroViewComponent.cs
new file mode 100644
index 0000000000..dc614009a7
--- /dev/null
+++ b/DotnetViewComponents/ViewComponents/HeroViewComponent.cs
@@ -0,0 +1,18 @@
+namespace DotnetViewComponents.ViewComponents
+{
+    using Microsoft.AspNetCore.Mvc;
+    using DotnetViewComponents.ViewModels;
+
+    public class HeroViewComponent : ViewComponent
+    {
+        public IViewComponentResult Invoke(
+            string imageSrc = null,
+            string heading = null,
+            string textContent = null)
+        { 
+            var model = new HeroViewModel(imageSrc, heading, textContent);
+
+            return View(model);
+        }
+    }
+}
diff --git a/DotnetViewComponents/ViewModels/HeroViewModel.cs b/DotnetViewComponents/ViewModels/HeroViewModel.cs
new file mode 100644
index 0000000000..3eeb7a4334
--- /dev/null
+++ b/DotnetViewComponents/ViewModels/HeroViewModel.cs
@@ -0,0 +1,23 @@
+namespace DotnetViewComponents.ViewModels
+{
+    public class HeroViewModel
+    {
+        public HeroViewModel(
+            string? imageSrc,
+            string? heading,
+            string? textContent)
+        {
+            ImageSrc = imageSrc;
+            Heading = heading;
+            TextContent = textContent;
+        }
+
+        public string? ImageSrc { get; set; }
+
+
+        public string? Heading { get; set;}
+
+
+        public string? TextContent { get; set;}
+    }
+}
diff --git a/DotnetViewComponents/Views/Shared/Components/Hero/Default.cshtml b/DotnetViewComponents/Views/Shared/Components/Hero/Default.cshtml
new file mode 100644
index 0000000000..1192841720
--- /dev/null
+++ b/DotnetViewComponents/Views/Shared/Components/Hero/Default.cshtml
@@ -0,0 +1,33 @@
+@using DotnetViewComponents.ViewModels
+@model HeroViewModel
+
+@{
+	var hasImage = !String.IsNullOrWhiteSpace(Model.ImageSrc);
+	var hasHeading = !String.IsNullOrWhiteSpace(Model.Heading);
+	var hasTextContent = !String.IsNullOrWhiteSpace(Model.TextContent);
+}
+<section class="nhsuk-hero @(hasImage ? "nhsuk-hero--image" : "") @(hasTextContent ? "nhsuk-hero--image-description" : "")" @(hasImage ? $"style=background-image:url({Model.ImageSrc})" : "")>
+	<div class="nhsuk-hero__overlay">
+		@if (hasHeading || hasTextContent)
+		{
+			<div class="nhsuk-width-container @(!hasImage ? "nhsuk-hero--border" : "")">
+				<div class="nhsuk-grid-row">
+					<div class="nhsuk-grid-column-two-thirds">
+						<div class="@(hasImage ? "nhsuk-hero-content" : "nhsuk-hero__wrapper")">
+							@if (hasHeading)
+							{
+								<h1 class="nhsuk-u-margin-bottom-3">@Model.Heading</h1>
+							}
+							@if (hasTextContent)
+							{
+								<p class="nhsuk-body-l nhsuk-u-margin-bottom-0">@Model.TextContent</p>
+							}
+
+							<span class="nhsuk-hero__arrow" aria-hidden="true"></span>
+						</div>
+					</div>
+				</div>
+			</div>
+		}
+	</div>
+</section>

From 8904ae7769ca24395819fff892469778995c8966 Mon Sep 17 00:00:00 2001
From: akdalin <dalin.akrasi@nhs.net>
Date: Thu, 29 May 2025 08:53:38 +0100
Subject: [PATCH 2/2] Make view component variables nullable. Add XML comment.

---
 .../ViewComponents/HeroViewComponent.cs         |  9 ++++++---
 .../ViewModels/HeroViewModel.cs                 | 17 +++++++++++++++--
 2 files changed, 21 insertions(+), 5 deletions(-)

diff --git a/DotnetViewComponents/ViewComponents/HeroViewComponent.cs b/DotnetViewComponents/ViewComponents/HeroViewComponent.cs
index dc614009a7..cfd47adcd8 100644
--- a/DotnetViewComponents/ViewComponents/HeroViewComponent.cs
+++ b/DotnetViewComponents/ViewComponents/HeroViewComponent.cs
@@ -3,12 +3,15 @@ namespace DotnetViewComponents.ViewComponents
     using Microsoft.AspNetCore.Mvc;
     using DotnetViewComponents.ViewModels;
 
+    /// <summary>
+    /// A ViewComponent that rendes a hero section.
+    /// </summary>
     public class HeroViewComponent : ViewComponent
     {
         public IViewComponentResult Invoke(
-            string imageSrc = null,
-            string heading = null,
-            string textContent = null)
+            string? imageSrc = null,
+            string? heading = null,
+            string? textContent = null)
         { 
             var model = new HeroViewModel(imageSrc, heading, textContent);
 
diff --git a/DotnetViewComponents/ViewModels/HeroViewModel.cs b/DotnetViewComponents/ViewModels/HeroViewModel.cs
index 3eeb7a4334..e1a79313c5 100644
--- a/DotnetViewComponents/ViewModels/HeroViewModel.cs
+++ b/DotnetViewComponents/ViewModels/HeroViewModel.cs
@@ -2,6 +2,12 @@ namespace DotnetViewComponents.ViewModels
 {
     public class HeroViewModel
     {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="HeroViewModel"/> class.
+        /// </summary>
+        /// <param name="imageSrc">The source URL of the hero image.</param>
+        /// <param name="heading">The heading text for the hero section.</param>
+        /// <param name="textContent">The main text content for the hero section.</param>
         public HeroViewModel(
             string? imageSrc,
             string? heading,
@@ -12,12 +18,19 @@ public HeroViewModel(
             TextContent = textContent;
         }
 
+        /// <summary>
+        /// Gets or sets the source URL of the hero image.
+        /// </summary>
         public string? ImageSrc { get; set; }
 
-
+        /// <summary>
+        /// Gets or sets the heading text for the hero section.
+        /// </summary>
         public string? Heading { get; set;}
 
-
+        /// <summary>
+        /// Gets or sets the main text content for the hero section.
+        /// </summary>
         public string? TextContent { get; set;}
     }
 }