From 7a325727b5e983f9ba7e3d3d76273f4a4eed23a8 Mon Sep 17 00:00:00 2001
From: Ari Stathopoulos <aristath@gmail.com>
Date: Fri, 3 Mar 2023 14:17:03 +0200
Subject: [PATCH] Use a select input instead of radios

---
 src/wp-admin/setup-config.php | 34 ++++++++++++----------------------
 1 file changed, 12 insertions(+), 22 deletions(-)

diff --git a/src/wp-admin/setup-config.php b/src/wp-admin/setup-config.php
index 5bb40c6fbe279..8f5c6a02ab6b7 100644
--- a/src/wp-admin/setup-config.php
+++ b/src/wp-admin/setup-config.php
@@ -224,14 +224,10 @@ function setup_config_display_header( $body_classes = array() ) {
 		<tr id="table-row-dbtype">
 			<th scope="row"><label for="dbtype"><?php _e( 'Database Engine' ); ?></label></th>
 			<td>
-				<div style="width:24ch;">
-					<input name="dbtype" id="dbtype-mysql" type="radio" value="mysql" aria-describedby="dbtype-desc" onclick="toggleDBType( this )" checked/>
-					<label for="dbtype-mysql"><?php _e( 'MySQL' ); ?></label>
-				</div>
-				<div style="width:24ch;">
-					<input name="dbtype" id="dbtype-sqlite" type="radio" value="sqlite" aria-describedby="dbtype-desc" onclick="toggleDBType( this )"/>
-					<label for="dbtype-sqlite"><?php _e( 'SQLite' ); ?></label>
-				</div>
+				<select name="dbtype" id="dbtype" aria-describedby="dbtype-desc" onChange="window.selectDBType( this );" style="width:26ch;">
+					<option value="mysql" selected><?php _e( 'MySQL' ); ?></option>
+					<option value="sqlite"><?php _e( 'SQLite' ); ?></option>
+				</select>
 			</td>
 			<td id="dbtype-desc"><?php _e( 'The type of database to use. SQLite is more suitable to personal blogs and smaller sites, while MySQL will allow your site to scale and grow easier.' ); ?></td>
 		</tr>
@@ -274,20 +270,14 @@ function setup_config_display_header( $body_classes = array() ) {
 	<p class="step"><input name="submit" type="submit" value="<?php echo htmlspecialchars( __( 'Submit' ), ENT_QUOTES ); ?>" class="button button-large" /></p>
 </form>
 <script>
-	window.toggleDBType = function( el ) {
-		if ( 'dbtype-sqlite' === el.id ) {
-			document.getElementById( 'table-row-dbname' ).style.display = 'none';
-			document.getElementById( 'table-row-uname' ).style.display = 'none';
-			document.getElementById( 'table-row-pwd' ).style.display = 'none';
-			document.getElementById( 'table-row-dbhost' ).style.display = 'none';
-			document.getElementById( 'table-row-prefix' ).style.display = 'none';
-		} else {
-			document.getElementById( 'table-row-dbname' ).style.display = 'table-row';
-			document.getElementById( 'table-row-uname' ).style.display = 'table-row';
-			document.getElementById( 'table-row-pwd' ).style.display = 'table-row';
-			document.getElementById( 'table-row-dbhost' ).style.display = 'table-row';
-			document.getElementById( 'table-row-prefix' ).style.display = 'table-row';
-		}
+	window.selectDBType = function( el ) {
+		console.log( el );
+		var display = 'sqlite' === el.value ? 'none' : 'table-row';
+		document.getElementById( 'table-row-dbname' ).style.display = display;
+		document.getElementById( 'table-row-uname' ).style.display = display;
+		document.getElementById( 'table-row-pwd' ).style.display = display;
+		document.getElementById( 'table-row-dbhost' ).style.display = display;
+		document.getElementById( 'table-row-prefix' ).style.display = display;
 	}
 </script>
 		<?php