From 95b52c47ddc5ca250c69aee4b7e3494b81a4baec Mon Sep 17 00:00:00 2001
From: Michael Green <84688932+michael-j-green@users.noreply.github.com>
Date: Wed, 6 Mar 2024 20:48:42 +1100
Subject: [PATCH] New library dialog content overruns the edges of the dialog
 (#323)

Closes #303

This fix is a clean up fix - will review functionality for 1.8.0
---
 .../wwwroot/pages/dialogs/librarynew.html     | 48 +++++++++----------
 .../wwwroot/pages/settings/settings.html      |  2 +-
 2 files changed, 23 insertions(+), 27 deletions(-)

diff --git a/gaseous-server/wwwroot/pages/dialogs/librarynew.html b/gaseous-server/wwwroot/pages/dialogs/librarynew.html
index 447a006..0f9c6d9 100644
--- a/gaseous-server/wwwroot/pages/dialogs/librarynew.html
+++ b/gaseous-server/wwwroot/pages/dialogs/librarynew.html
@@ -1,34 +1,30 @@
-<div style="padding-top: 5px;">
-    <strong>New Library</strong>
-</div>
+<table style="width: 98%; margin-top: 15px; margin-bottom: 15px;">
+    <tr>
+        <th style="width: 20%;">Name</th>
+        <td style="width: 80%;"><input type="text" id="newlibrary_name" style="width: 98%;" /></td>
+    </tr>
+    <tr>
+        <th>Default Platform</th>
+        <td><select id="newlibrary_defaultplatform" style="width: 100%;"></select></td>
+    </tr>
+    <tr>
+        <th>Path</th>
+        <td><input type="text" id="newlibrary_path" style="width: 98%;" /></td>
+    </tr>
+</table>
 
-<div style="width: 300px;">
-    <table style="width: 98%; margin-top: 15px; margin-bottom: 15px;">
-        <tr>
-            <th>Name</th>
-            <td><input type="text" id="newlibrary_name" style="width: 95%;" /></td>
-        </tr>
-        <tr>
-            <th>Default Platform</th>
-            <td><select id="newlibrary_defaultplatform" style="width: 100%;"></select></td>
-        </tr>
-        <tr>
-            <th>Path</th>
-            <td><input type="text" id="newlibrary_path" style="width: 95%;" /></td>
-        </tr>
-    </table>
-
-    <div style="width: 100%; text-align: right;">
-        <div style="display: inline-block; margin-right: 20px;">
-            <button value="OK" onclick="newLibrary();">OK</button>
-        </div>
-        <div style="display: inline-block;">
-            <button value="Cancel" onclick="closeSubDialog();">Cancel</button>
-        </div>
+<div style="width: 100%; text-align: right; margin-top: 160px;">
+    <div style="display: inline-block; margin-right: 20px;">
+        <button value="OK" onclick="newLibrary();">OK</button>
+    </div>
+    <div style="display: inline-block;">
+        <button value="Cancel" onclick="closeDialog();">Cancel</button>
     </div>
 </div>
 
 <script type="text/javascript">
+    document.getElementById('modal-heading').innerHTML = "New Library";
+
     $('#newlibrary_defaultplatform').select2({
         minimumInputLength: 3,
         ajax: {
diff --git a/gaseous-server/wwwroot/pages/settings/settings.html b/gaseous-server/wwwroot/pages/settings/settings.html
index 6e0a9d9..93e6a06 100644
--- a/gaseous-server/wwwroot/pages/settings/settings.html
+++ b/gaseous-server/wwwroot/pages/settings/settings.html
@@ -6,7 +6,7 @@
 <table id="settings_libraries" class="romtable" style="width: 100%;" cellspacing="0">
 
 </table>
-<div style="text-align: right;"><button id="settings_newlibrary" onclick="showSubDialog('librarynew');">New Library</button></div>
+<div style="text-align: right;"><button id="settings_newlibrary" onclick="showDialog('librarynew');">New Library</button></div>
 
 <h2>Advanced Settings</h2>
 <p><strong>Warning</strong> Do not modify the below settings unless you know what you're doing.</p>