Add Height property to ScrollablePanelControl

Author: nickprotop

SharpConsoleUI/Builders/ScrollablePanelBuilder.cs

@@ -37,6 +37,7 @@ public sealed class ScrollablePanelBuilder
 	private Margin _margin = new(0, 0, 0, 0);
 	private bool _visible = true;
 	private int? _width;
+	private int? _height;
 	private string? _name;
 	private object? _tag;
 	private StickyPosition _stickyPosition = StickyPosition.None;
@@ -255,6 +256,17 @@ public ScrollablePanelBuilder WithWidth(int width)
 		return this;
 	}
 
+	/// <summary>
+	/// Sets the height
+	/// </summary>
+	/// <param name="height">The height</param>
+	/// <returns>The builder for chaining</returns>
+	public ScrollablePanelBuilder WithHeight(int height)
+	{
+		_height = height;
+		return this;
+	}
+
 	/// <summary>
 	/// Sets the control name for FindControl queries
 	/// </summary>
@@ -362,6 +374,7 @@ public ScrollablePanelControl Build()
 			Margin = _margin,
 			Visible = _visible,
 			Width = _width,
+			Height = _height,
 			Name = _name,
 			Tag = _tag,
 			StickyPosition = _stickyPosition,

SharpConsoleUI/Controls/ScrollablePanelControl.cs

@@ -61,6 +61,7 @@ public class ScrollablePanelControl : IWindowControl, IInteractiveControl, IFocu
 		private StickyPosition _stickyPosition = StickyPosition.None;
 		private bool _visible = true;
 		private int? _width;
+		private int? _height;
 
 		// IContainer properties
 		private Color _backgroundColor = Color.Black;
@@ -207,7 +208,7 @@ public bool AutoScroll
 		#region IWindowControl Implementation
 
 		/// <inheritdoc/>
-		public int? ContentHeight => null;  // Fill available space
+		public int? ContentHeight => _height;
 
 		/// <inheritdoc/>
 		public int? ContentWidth => _width;
@@ -289,6 +290,13 @@ public int? Width
 			set { _width = value; Container?.Invalidate(true); }
 		}
 
+		/// <inheritdoc/>
+		public int? Height
+		{
+			get => _height;
+			set => PropertySetterHelper.SetDimensionProperty(ref _height, value, Container);
+		}
+
 		/// <inheritdoc/>
 		public System.Drawing.Size GetLogicalContentSize()
 		{
@@ -1137,12 +1145,23 @@ public LayoutSize MeasureDOM(LayoutConstraints constraints)
 			int width = _width ?? constraints.MaxWidth;
 			int availableWidth = Math.Max(1, width - _margin.Left - _margin.Right);
 
-			// Pass the actual available width to CalculateContentHeight
-			int height = CalculateContentHeight(availableWidth);
+			// Determine height
+			int height;
+			if (_height.HasValue)
+			{
+				// Explicit height set - use it directly
+				height = _height.Value;
+			}
+			else
+			{
+				// No explicit height - calculate from content
+				int contentHeight = CalculateContentHeight(availableWidth);
+				height = contentHeight + _margin.Top + _margin.Bottom;
+			}
 
 			return new LayoutSize(
 				Math.Clamp(width + _margin.Left + _margin.Right, constraints.MinWidth, constraints.MaxWidth),
-				Math.Clamp(height + _margin.Top + _margin.Bottom, constraints.MinHeight, constraints.MaxHeight)
+				Math.Clamp(height, constraints.MinHeight, constraints.MaxHeight)
 			);
 		}