From 4931d00990c707d90e5fb913c20c6a74ef2e30fc Mon Sep 17 00:00:00 2001
From: Zack Spear <hi@zackspear.com>
Date: Tue, 20 May 2025 17:25:12 -0700
Subject: [PATCH] fix: enhance layout consistency in PHPsettings.page and
 PHPsettings.css

- Updated PHPsettings.page to wrap buttons in a div for improved layout and added a new section for log management.
- Modified PHPsettings.css to define a new class for the PHP settings description, ensuring better alignment and responsiveness.
- This change continues the effort to enhance visual structure across the plugin.
---
 emhttp/plugins/dynamix/PHPsettings.page       | 20 ++++++++++++++++---
 emhttp/plugins/dynamix/sheets/PHPsettings.css |  7 +++++--
 2 files changed, 22 insertions(+), 5 deletions(-)

diff --git a/emhttp/plugins/dynamix/PHPsettings.page b/emhttp/plugins/dynamix/PHPsettings.page
index f9876564f..d8eff8424 100644
--- a/emhttp/plugins/dynamix/PHPsettings.page
+++ b/emhttp/plugins/dynamix/PHPsettings.page
@@ -22,7 +22,7 @@ $conf = file_exists($ini) ? parse_ini_file($ini) : [];
 if (!file_exists($log)) touch($log);
 ?>
 
-<div markdown="1" style="width:43%;margin-bottom:40px;padding:5px 15px;border:solid 1px">
+<div markdown="1" class="php-settings-description">
 :php_settings_plug:
 This utility is used for development purposes only and allows Plugin Authors to verify their PHP code by enabling different levels of PHP error reporting.
 
@@ -42,6 +42,17 @@ under normal running conditions.
 <input type="hidden" name="display_startup_errors" value="0">
 <input type="hidden" name="display_errors" value="<?=_var($conf,'display_errors',0)?>">
 <input type="hidden" name="log_errors" value="1">
+
+&nbsp;
+: <div class="inline-block">
+    <input type="button" value="_(PHP Info)_" onclick="PHPinfo()">
+    <input type="button" id="clearlog" value="_(Clear Log)_" onclick="clearLog()" <?= filesize($log) > 0 ? '' : ' disabled'?>>
+    <input type="button" id="viewlog" value="_(View Log)_" onclick="viewLog()" disabled>
+    <span class="inline-block">
+      _(LOG size)_:&nbsp;<span id="logsize"></span>
+    </span>
+  </div>
+
 _(Error reporting level)_:
 : <select name="error_reporting" onchange="toggleScreen(this.selectedIndex)">
   <?=mk_option(_var($conf,'error_reporting'), strval(E_ALL & ~E_NOTICE & ~E_WARNING & ~E_DEPRECATED), "_(Default)_");?>
@@ -53,9 +64,12 @@ _(Error reporting level)_:
   </select>
 
 &nbsp;
-: <input type="checkbox" id="screenlog" onclick="screenLog()" <?=_var($conf,'display_errors',0)?'checked':''?> disabled><span id="screentext" class="dim">_(Show errors on screen)_</span>
+: <label for="screenlog">
+    <input type="checkbox" id="screenlog" onclick="screenLog()" <?=_var($conf,'display_errors',0)?'checked':''?> disabled>
+    <span id="screentext" class="dim">_(Show errors on screen)_</span>
+  </label>
 
-<input type="button" value="_(PHP Info)_" onclick="PHPinfo()"><input type="button" id="clearlog" value="_(Clear Log)_" onclick="clearLog()"<?=filesize($log)>0?'':' disabled'?>><input type="button" id="viewlog" value="_(View Log)_" onclick="viewLog()" disabled>_(LOG size)_:&nbsp;<span id="logsize"></span>
+&nbsp;
 : <span class="inline-block">
     <input type="submit" value="_(Apply)_" disabled>
     <input type="button" value="_(Done)_" onclick="done()">
diff --git a/emhttp/plugins/dynamix/sheets/PHPsettings.css b/emhttp/plugins/dynamix/sheets/PHPsettings.css
index 0a1d05f5c..182c34334 100644
--- a/emhttp/plugins/dynamix/sheets/PHPsettings.css
+++ b/emhttp/plugins/dynamix/sheets/PHPsettings.css
@@ -2,9 +2,12 @@ div#templateWindow,
 div#dialogWindow {
     display: none;
 }
+
 span.dim {
     opacity: 0.2;
 }
-#screenlog {
-    margin-left: 0;
+
+.php-settings-description {
+    max-width: 100ch;
+    margin: 0 auto;
 }