From a65c442db925ff73abc8d856673c1ceaed494f05 Mon Sep 17 00:00:00 2001
From: Zack Spear <hi@zackspear.com>
Date: Wed, 21 May 2025 16:52:34 -0700
Subject: [PATCH] fix: enhance layout consistency in PluginInstall.page and
 default-base.css

- Added utility classes for width, height, and overflow to improve layout structure in PluginInstall.page.
- Removed the obsolete Plugins.css file as its styles are no longer needed.
- This change continues the effort to enhance visual consistency across the plugin.
---
 .../dynamix.plugin.manager/PluginInstall.page |  4 +--
 .../dynamix.plugin.manager/sheets/Plugins.css |  8 -----
 .../plugins/dynamix/styles/default-base.css   | 29 +++++++++++++++++++
 3 files changed, 31 insertions(+), 10 deletions(-)
 delete mode 100644 emhttp/plugins/dynamix.plugin.manager/sheets/Plugins.css

diff --git a/emhttp/plugins/dynamix.plugin.manager/PluginInstall.page b/emhttp/plugins/dynamix.plugin.manager/PluginInstall.page
index fdbd8c596..4de43cdb5 100644
--- a/emhttp/plugins/dynamix.plugin.manager/PluginInstall.page
+++ b/emhttp/plugins/dynamix.plugin.manager/PluginInstall.page
@@ -65,7 +65,7 @@ Tag="download"
 
 :plugin_install_help:
 
-<div>
+<div class="max-w-20 mx-auto">
   <p><strong>_(Select local plugin file)_</strong></p>
-  <div id="plugin_tree" class="textarea"></div>
+  <div id="plugin_tree" class="textarea h-50 overflow-y-scroll scrollbar-thin"></div>
 </div>
diff --git a/emhttp/plugins/dynamix.plugin.manager/sheets/Plugins.css b/emhttp/plugins/dynamix.plugin.manager/sheets/Plugins.css
deleted file mode 100644
index ac69352cd..000000000
--- a/emhttp/plugins/dynamix.plugin.manager/sheets/Plugins.css
+++ /dev/null
@@ -1,8 +0,0 @@
-#plugin_tree {
-    width: 33%;
-    height: 200px;
-    overflow-y: scroll;
-}
-table tbody td {
-    line-height: normal;
-}
\ No newline at end of file
diff --git a/emhttp/plugins/dynamix/styles/default-base.css b/emhttp/plugins/dynamix/styles/default-base.css
index 65db00d94..866b826d7 100644
--- a/emhttp/plugins/dynamix/styles/default-base.css
+++ b/emhttp/plugins/dynamix/styles/default-base.css
@@ -1870,6 +1870,35 @@ span#wlan0 {
 .justify-self-stretch {
     justify-self: stretch;
 }
+
+.w-full {
+    width: 100%;
+}
+/* matches Tailwind scale */
+.max-w-20 {
+    max-width: 80rem; /* 1rem = 4. 80/4 = 20. 1rem = 10px with current font-size */
+}
+
+.mx-auto {
+    margin-left: auto;
+    margin-right: auto;
+}
+
+.h-50 {
+    height: 20rem;
+}
+
+.overflow-y-auto {
+    overflow-y: auto;
+}
+
+.overflow-y-scroll {
+    overflow-y: scroll;
+}
+
+.scrollbar-thin {
+    scrollbar-width: thin;
+}
 /*
  * Using CSS Nesting, to narrow down the scope of the styles to the .Theme--sidebar class.
  * This allows us to have default-azure & default-gray set css variables