From 0c268f3b56604367ca5dfe887d07774cdbe553e5 Mon Sep 17 00:00:00 2001
From: Jake Vanderwerf <get@jakevanderwerf.ca>
Date: Tue, 10 Feb 2026 23:34:02 +0000
Subject: [PATCH] =minor tweaks to form.php

---
 inc/meta/Form.php |   91 +++++++++++++++++++++++++++++----------------
 1 files changed, 58 insertions(+), 33 deletions(-)

diff --git a/inc/meta/Form.php b/inc/meta/Form.php
index 7ba1458..263fda2 100644
--- a/inc/meta/Form.php
+++ b/inc/meta/Form.php
@@ -87,7 +87,7 @@
 	// Helper Methods
 	// ─────────────────────────────────────────────────────────────
 
-	protected static function fieldWrap(string $name, string $content, array $config): string
+	public static function fieldWrap(string $name, string $content, array $config): string
 	{
 		$classes = static::buildClasses($config);
 		$datasets = static::buildDatasets($config);
@@ -107,6 +107,8 @@
 		$output .= static::buildLabel($name, $config);
 		if (!array_key_exists('skipInput', $config)) {
 			$output .= static::buildInput($content);
+		} else {
+			$output .= $content;
 		}
 
 		$output .= static::buildHint($config);
@@ -208,7 +210,7 @@
 			return '';
 		}
 
-		protected static function buildInput(string $content):string
+		public static function buildInput(string $content):string
 		{
 			return sprintf(
 				'<div class="field-input-wrapper">
@@ -570,30 +572,47 @@
 
 	protected static function renderRadio(string $name, mixed $value, array $config): string
 	{
-		$options = $config['options'] ?? [];
+		$options    = $config['options'] ?? [];
+		$inputClass = !empty($config['inputClass']) ? ' class="' . esc_attr($config['inputClass']) . '"' : '';
+		$idPrefix   = $config['idPrefix'] ?? '';
 
 		$radios = sprintf(
 			'<fieldset>
-			<legend>%s%s</legend>',
+        <legend>%s%s</legend>',
 			array_key_exists('label', $config) ? esc_html($config['label']) : 'Select an option',
-			array_key_exists('required', $config) && $config['required']===true ? '<span class="required" aria-label="required">*</span>' : ''
+			array_key_exists('required', $config) && $config['required'] === true
+				? '<span class="required" aria-label="required">*</span>' : ''
 		);
 
-		foreach ($options as $optValue => $optLabel) {
+		foreach ($options as $optValue => $optConfig) {
+			if (is_array($optConfig)) {
+				$optLabel    = $optConfig['label'] ?? $optValue;
+				$optIcon     = $optConfig['icon'] ?? null;
+				$optDisabled = !empty($optConfig['disabled']) ? ' disabled' : '';
+			} else {
+				$optLabel    = $optConfig;
+				$optIcon     = null;
+				$optDisabled = '';
+			}
+
+			$labelContent = $optIcon
+				? jvbDashIcon($optIcon)
+				: '<span>' . esc_html($optLabel) . '</span>';
+
+			$optId = esc_attr($idPrefix . $name . '-' . $optValue);
+
 			$radios .= sprintf(
-				'
-                    <input type="radio" name="%s" id="%s-%s" value="%s"%s />
-				<label class="radio-option" for="%s-%s">
-                    <span>%s</span>
-                </label>',
+				'<input type="radio" name="%s" id="%s" value="%s"%s%s%s />
+            <label class="radio-option" for="%s" title="%s">%s</label>',
 				esc_attr($name),
-				esc_attr($name),
-				$optValue,
+				$optId,
 				esc_attr($optValue),
-				checked($value, $optValue,false),
-				esc_attr($name),
-				$optValue,
-				esc_html($optLabel)
+				checked($value, $optValue, false),
+				$optDisabled,
+				$inputClass,
+				$optId,
+				esc_html($optLabel),
+				$labelContent
 			);
 		}
 
@@ -1170,7 +1189,10 @@
 
 		$input = sprintf(
 			'<div class="row btw">
-			<label for="%s-autocomplete">%s<span>%s</span></label>',
+    <input type="hidden" name="%s" value="%s">
+    <label for="%s-autocomplete">%s<span>%s</span></label>',
+			esc_attr($name),
+			esc_attr(!empty($ids) ? implode(',', $ids) : ''),
 			esc_attr($name),
 			jvbIcon($icon),
 			esc_html($config['label']),
@@ -1183,15 +1205,18 @@
 		}
 		$plural = static::getPlural($config);
 		$input .= sprintf(
-			'<div class="selected-item row" role="region" aria-label="Selected %s"></div>',
+			'<div class="selected-items row" role="region" aria-label="Selected %s"></div>',
 			$plural[1]??''
 		);
 
+		$input .= '</div>'; //Close the first div.row.btw
+
+		$config['type'] = 'selector';
 		unset($config['label']);
 		unset($config['description']);
 		unset($config['hint']);
 		$config['skipInput'] = true;
-		return static::fieldWrap($containerId, $input, $config);
+		return static::fieldWrap($name, $input, $config);
 	}
 
 		protected static function getPlural(array $config):array
@@ -1257,7 +1282,7 @@
 						$plural = $taxonomy->labels->name;
 					}
 					$attr = sprintf(
-						' data-taxonomy="%s" data-single="%s" data-plural="%s',
+						' data-taxonomy="%s" data-single="%s" data-plural="%s"',
 						$config['taxonomy'],
 						$single,
 						$plural
@@ -1276,7 +1301,7 @@
 						$plural = $postType->labels->name;
 					}
 					$attr = sprintf(
-						' data-content="%s" data-single="%s" data-plural="%s',
+						' data-content="%s" data-single="%s" data-plural="%s"',
 						$config['content'],
 						$single,
 						$plural
@@ -1296,7 +1321,7 @@
 						$plural = 'Users';
 					}
 					$attr = sprintf(
-						' data-user="%s" data-single="%s" data-plural="%s',
+						' data-user="%s" data-single="%s" data-plural="%s"',
 						$config['user'],
 						$single,
 						$plural
@@ -1324,7 +1349,7 @@
 				$dataAttrs[] = 'data-selected="'.esc_attr(implode(',',$selected)).'"';
 			}
 			if ($config['autocomplete']) {
-				$dataAttrs[] = 'autocomplete';
+				$dataAttrs[] = 'data-autocomplete';
 			}
 			if (array_key_exists('hidden', $config) && $config['hidden']) {
 				$dataAttrs[] = 'hidden';
@@ -1341,15 +1366,15 @@
 				jvbIcon('plus-square')
 			);
 		}
-		protected static function buildSelectorAutocomplete(string $name, array $config):string
-		{
-			return sprintf(
-			'<input type="hidden" id="%s-autocomplete" autocomplete="off" data-ignore data-autocomplete>
-				<p class="message" hidden aria-live="polite">{ <span>Loading items</span> }</p>
-				<div class="auto-wrapper" hidden><ul class="search-results"></ul><button class="submit-term" hidden data-ignore><strong>Create: </strong> "<span></span>"</button></div>',
-				$name
-			);
-		}
+	protected static function buildSelectorAutocomplete(string $name, array $config): string
+	{
+		return sprintf(
+			'<input type="search" id="%s-autocomplete" autocomplete="off" data-ignore data-autocomplete>
+        <p class="message" hidden aria-live="polite">{ <span>Loading items</span> }</p>
+        <div class="auto-wrapper" hidden><ul class="search-results"></ul><button class="submit-term" hidden data-ignore><strong>Create: </strong> "<span></span>"</button></div>',
+			esc_attr($name)
+		);
+	}
 
 	protected static function renderTaxonomy(string $name, mixed $value, array $config): string
 	{

--
Gitblit v1.10.0