Browse code

Making admin settings sections look even less jank

Clarissa Walker authored on 2026/07/26 17:29:21
Showing 1 changed files
... ...
@@ -310,10 +310,10 @@ else if(isset($_POST['submit'])) {
310 310
 		$$var = stripslashes($val );
311 311
 	}
312 312
 
313
-	$output .= "<form method='POST' class='tblborder' style='width:30%; margin:0 auto' enctype='multipart/form-data' action='".((isset($action) && $action == "settings") ? "admin.php?action=settings" : $_SERVER['PHP_SELF']."?step=".$_GET['step'])."&amp;sect=$sect'>";
313
+	$output .= "<br><form method='POST' class='tblborder' style='width: 45%; margin: 0 auto;' enctype='multipart/form-data' action='".((isset($action) && $action == "settings") ? "admin.php?action=settings" : $_SERVER['PHP_SELF']."?step=".$_GET['step'])."&amp;sect=$sect'>";
314 314
 	if($sect == "main") {
315 315
 		$output .= "<div class='sectionheader'>"._SITEINFO."</div>
316
-		<table class='acp'>
316
+		<table class='acp' style='margin: 0 auto;'>
317 317
 			<tr>
318 318
 				<td><label for='newsitekey'>"._SITEKEY.":</label></td><td><input type='text' class='textbox' name='newsitekey' value='".SITEKEY."'> <a href='#' class='pophelp'>[?]<span>"._HELP_SITEKEY."</span></a></td>
319 319
 			</tr>
... ...
@@ -356,7 +356,7 @@ else if(isset($_POST['submit'])) {
356 356
 	}
357 357
 	else if($sect == "submissions") {
358 358
 		$output .= "<div class='sectionheader'>"._SUBMISSIONSETTINGS."</div>
359
-		<table class='acp'>
359
+		<table class='acp' style='margin: 0 auto;'>
360 360
 			<tr>
361 361
 				<td><label for='newsubmissionsoff'>"._NOSUBS.":</label></td><td><select name='newsubmissionsoff'>
362 362
 				<option value='1'".($submissionsoff == "1" ? " selected" : "").">"._YES."</option>
... ...
@@ -396,8 +396,8 @@ else if(isset($_POST['submit'])) {
396 396
 			</tr>
397 397
 			<tr>
398 398
 				<td colspan='2'><fieldset style='margin: 0 auto;'><legend>"._IMAGESIZE." <a href='#' class='pophelp'>[?]<span>"._HELP_IMAGESIZE."</span></a></legend>
399
-				<label for='newimageheight'>"._MAXHEIGHT.":</label> &nbsp; &nbsp; <input  type='text' class='textbox=' name='newimageheight' value='$imageheight' size='5'> <br />
400
-				<label for='newimagewidth'>"._MAXWIDTH.":</label> &nbsp; &nbsp; &nbsp; <input  type='text' class='textbox=' name='newimagewidth' value='$imagewidth' size='5'></fieldset></td>
399
+				<label for='newimageheight'>"._MAXHEIGHT.":</label> &nbsp; &nbsp; &nbsp; <input  type='text' class='textbox=' name='newimageheight' value='$imageheight' size='5'> <br />
400
+				<label for='newimagewidth'>"._MAXWIDTH.":</label> &nbsp; &nbsp; &nbsp; &nbsp; <input  type='text' class='textbox=' name='newimagewidth' value='$imagewidth' size='5'></fieldset></td>
401 401
 			</tr>
402 402
 			<tr>
403 403
 				<td><label for='newstore'>"._HOWSTORE.":</label></td><td><select name='newstore' onChange='if (this.disabled) this.selectedIndex=0' disabled>
... ...
@@ -415,7 +415,7 @@ else if(isset($_POST['submit'])) {
415 415
 	}
416 416
 	else if($sect == "sitesettings") {
417 417
 		$output .= "<div class='sectionheader'>"._SITESETTINGS."</div>
418
-		<table class='acp'>
418
+		<table class='acp' style='margin: 0 auto;'>
419 419
 			<tr>
420 420
 				<td><label for='newtinyMCE'>"._USETINYMCE. ": </label></td><td><select name='newtinyMCE'>
421 421
 				<option value=\"2\"" . ($tinyMCE == 2 ? " selected" : "") . ">TinyMce4</option>
... ...
@@ -475,7 +475,7 @@ else if(isset($_POST['submit'])) {
475 475
 				"d.m.y", "Y.m.d", "m.d.Y", "d-m-y", "m-d-y", "M d Y", "M d, Y", "F d Y", "F d, Y");
476 476
 		$defaulttimes = array("h:i a", "h:i A", "H:i", "g:i a", "g:i A", "G:i", "h:i:s a", "H:i:s", "g:i:s a", "g:i:s A", "G:i:s");
477 477
 		$output .= "<div class='sectionheader'>"._DISPLAYSETTINGS."</div>
478
-		<table class='acp'>
478
+		<table class='acp' style='margin: 0 auto;'>
479 479
 			<tr>
480 480
 				<td><label for='newdateformat'>"._DATEFORMAT.":</label></td><td><select name='newdateformat'><option value=''>"._SELECTONE."</option>";
481 481
 		foreach($defaultdates as $date) {
... ...
@@ -536,7 +536,7 @@ else if(isset($_POST['submit'])) {
536 536
 	}
537 537
 	else if($sect == "reviews") {
538 538
 		$output .= "<div class='sectionheader'>"._REVIEWSETTINGS."</div>
539
-		<table class='acp'>
539
+		<table class='acp' style='margin: 0 auto;'>
540 540
 			<tr>
541 541
 				<td><label for='newreviewsallowed'>"._ONREVIEWS.":</label></td><td><select name='newreviewsallowed'>
542 542
 				<option value='1'".($reviewsallowed == "1" ? " selected" : "").">"._YES."</option>
... ...
@@ -572,7 +572,7 @@ else if(isset($_POST['submit'])) {
572 572
 	}
573 573
 	else if($sect == "useropts") {
574 574
 		$output .= "<div class='sectionheader'>"._USERSETTINGS."</div>
575
-		<table class='acp'>
575
+		<table class='acp' style='margin: 0 auto;'>
576 576
 			<tr>
577 577
 				<td><label for='newalertson'>"._ALERTSON.":</label></td><td><select name='newalertson'>
578 578
 				<option value='1'".($alertson == "1" ? " selected" : "").">"._YES."</option>
... ...
@@ -599,7 +599,7 @@ else if(isset($_POST['submit'])) {
599 599
 	}
600 600
 	else if($sect == "email") {
601 601
 		$output .= "<div class='sectionheader'>"._EMAILSETTINGS."</div>
602
-		<table class='acp'>
602
+		<table class='acp' style='margin: 0 auto;'>
603 603
 			<tr>
604 604
 				<td><label for='newsmtp_host'>"._SMTPHOST.":</label></td><td><input name='newsmtp_host' type='text' value='$smtp_host'> <a href='#' class='pophelp'>[?]<span>"._HELP_SMTPHOST."</span></a></td>
605 605
 		</tr>
... ...
@@ -619,20 +619,19 @@ else if(isset($_POST['submit'])) {
619 619
 				</select> <a href='#' class='pophelp'>[?]<span>"._HELP_SMTPSECURE."</span></a></td></tr>";
620 620
 		$output .= 	write_message(_SMTPOFF);
621 621
 	}
622
-	$output .= "<tr><td colspan='2'><div align='center'><input type='submit' id='submit' class='button' name='submit' value='"._SUBMIT."'></div></form></td></tr></table>";
622
+	$output .= "<tr><td colspan='2'><div align='center'><br><input type='submit' id='submit' class='button' name='submit' value='"._SUBMIT."'><p></div></form></td></tr></table>";
623 623
 	// The SMTP test button lives OUTSIDE the main settings form (nested forms are
624 624
 	// invalid HTML). It is its own POST mini-form -- a submit button, not a GET
625 625
 	// link -- so the outbound SMTP connection can't be triggered by a plain link
626 626
 	// or a browser prefetch. The existing testsmtp handler posts to the same URL.
627 627
 	if($sect == "email") {
628 628
 		$output .= "<div class='sectionheader'>"._TESTSMTP_HEADER."</div>
629
-		<ul>
630
-			<li>
629
+		<div align='center'>
631 630
 				<form method='POST' action='admin.php?action=settings&amp;sect=email' style='display:inline'>
632 631
 					<input type='submit' class='button' name='testsmtp' value='"._TESTSMTP."'>
633 632
 				</form>
634 633
 				<a href='#' class='pophelp'>[?]<span>"._HELP_TESTSMTP."</span></a>
635
-			</li>
636
-		</ul>";
634
+			<p>
635
+		</div>";
637 636
 	}
638 637
 ?>