Updating settings UI

Author: Sahasrara

Assets/GameScriptSettings.asset

@@ -16,7 +16,6 @@ MonoBehaviour:
   InitialConversationPool: 1
   PreventSingleNodeChoices: 1
   DatabasePath: D:/Work/GameScriptUnity/Database/TestDB
-  DatabaseVersion: 0.0.0
-  GeneratedPath: D:/Work/GameScriptUnity/Assets/GameScript
-  GameDataPath: D:/Work/GameScriptUnity/Assets/StreamingAssets/GameScript
-  GameDataPathRelative: /GameScript
+  DatabaseVersion: 
+  GeneratedPath: 
+  GameDataPath: /Users/eric/Work/GameScriptUnity/Assets/StreamingAssets/GameScript

Packages/studio.shortsleeve.gamescriptunity/Editor/Menu/SettingsEditor.cs

@@ -119,12 +119,46 @@ public override VisualElement CreateInspectorGUI()
             databasePathField.SetEnabled(false);
 
             // Select Editor Database Button
+            Button databaseSelectButton = new();
+            databaseSelectButton.name = "SelectDatabase";
+            databaseSelectButton.text = "Select Database";
+            databaseSelectButton.tooltip =
+                "Select a database file to use for importing conversation data.";
+
+            // Import Database Button
             Button databaseImportButton = new();
             databaseImportButton.name = "ImportDatabase";
             databaseImportButton.text = "Import Database";
             databaseImportButton.tooltip =
-                "Select a database file to import. This will prepare "
+                "Import the selected database. This will prepare "
                 + "all of your conversation data for runtime use and may take some time.";
+
+            // Wire up Select Database Button callback
+            databaseSelectButton.clickable.clicked += () =>
+            {
+                string absoluteDbPath = EditorUtility.OpenFilePanel("Select Database", "", "");
+                if (string.IsNullOrEmpty(absoluteDbPath))
+                    return;
+
+                // Convert to relative and save
+                string projectRoot = Path.GetFullPath(Path.Combine(Application.dataPath, ".."));
+                string relativeDbPath = GetRelativePath(projectRoot, absoluteDbPath);
+                serializedObject.FindProperty("DatabasePath").stringValue = relativeDbPath;
+                serializedObject.ApplyModifiedProperties();
+
+                // Update display with absolute path
+                databasePathField.value = absoluteDbPath;
+
+                // Revalidate to update Import button state
+                ValidatePathsAndUpdateImportButton(
+                    generatedPathField,
+                    runtimeDatabasePath,
+                    databasePathField,
+                    databaseImportButton
+                );
+            };
+
+            // Wire up Import Database Button callback
             databaseImportButton.clickable.clicked += () =>
             {
                 // Make sure import isn't in progress
@@ -138,22 +172,9 @@ public override VisualElement CreateInspectorGUI()
                     return;
                 }
 
-                // Select database file
-                string absoluteDbPath = EditorUtility.OpenFilePanel("Import Database", "", "");
-                if (string.IsNullOrEmpty(absoluteDbPath))
-                    return;
-
-                // Convert to relative and save
-                string projectRoot = Path.GetFullPath(Path.Combine(Application.dataPath, ".."));
-                string relativeDbPath = GetRelativePath(projectRoot, absoluteDbPath);
-                serializedObject.FindProperty("DatabasePath").stringValue = relativeDbPath;
-                serializedObject.ApplyModifiedProperties();
-
-                // Update display
-                databasePathField.value = absoluteDbPath;
-
                 // Load database using absolute paths
                 Settings settings = (Settings)serializedObject.targetObject;
+                string absoluteDbPath = settings.GetAbsoluteDatabasePath();
                 string gameScriptOutputPath = settings.GetAbsoluteGeneratedPath();
                 string conversationOutputPath = settings.GetAbsoluteGameDataPath();
                 DatabaseImporter.ImportDatabase(
@@ -188,6 +209,7 @@ public override VisualElement CreateInspectorGUI()
                     ValidatePathsAndUpdateImportButton(
                         generatedPathField,
                         runtimeDatabasePath,
+                        databasePathField,
                         databaseImportButton
                     );
                 }
@@ -213,6 +235,7 @@ public override VisualElement CreateInspectorGUI()
                     ValidatePathsAndUpdateImportButton(
                         generatedPathField,
                         runtimeDatabasePath,
+                        databasePathField,
                         databaseImportButton
                     );
                 }
@@ -234,6 +257,7 @@ public override VisualElement CreateInspectorGUI()
             editorSettingsFoldout.Add(databaseHeader);
             editorSettingsFoldout.Add(databasePathField);
             editorSettingsFoldout.Add(databaseVersionField);
+            editorSettingsFoldout.Add(databaseSelectButton);
             editorSettingsFoldout.Add(databaseImportButton);
             #endregion
 
@@ -255,6 +279,7 @@ public override VisualElement CreateInspectorGUI()
             ValidatePathsAndUpdateImportButton(
                 generatedPathField,
                 runtimeDatabasePath,
+                databasePathField,
                 databaseImportButton
             );
 
@@ -268,6 +293,7 @@ public override VisualElement CreateInspectorGUI()
                     ValidatePathsAndUpdateImportButton(
                         generatedPathField,
                         runtimeDatabasePath,
+                        databasePathField,
                         databaseImportButton
                     );
                 }
@@ -282,6 +308,7 @@ public override VisualElement CreateInspectorGUI()
                     ValidatePathsAndUpdateImportButton(
                         generatedPathField,
                         runtimeDatabasePath,
+                        databasePathField,
                         databaseImportButton
                     );
                 }
@@ -331,11 +358,12 @@ private void OnDbVersionChanged(TextField databaseVersionField, string newVersio
         }
 
         /// <summary>
-        /// Validates both required paths and updates the Import button state and field highlighting
+        /// Validates all required paths and updates the Import button state and field highlighting
         /// </summary>
         private void ValidatePathsAndUpdateImportButton(
             TextField generatedPathField,
             TextField runtimeDatabasePath,
+            TextField databasePathField,
             Button databaseImportButton
         )
         {
@@ -353,6 +381,10 @@ Button databaseImportButton
                 requireExists: false
             );
 
+            // Database file must exist
+            bool databasePathValid = !string.IsNullOrEmpty(databasePathField.value)
+                && File.Exists(databasePathField.value);
+
             // Update field highlighting
             if (generatedPathValid)
             {
@@ -372,8 +404,17 @@ Button databaseImportButton
                 SetElementWarning(runtimeDatabasePath);
             }
 
-            // Enable Import button only if both paths are valid
-            databaseImportButton.SetEnabled(generatedPathValid && gameDataPathValid);
+            if (databasePathValid)
+            {
+                UnsetElementWarning(databasePathField);
+            }
+            else
+            {
+                SetElementWarning(databasePathField);
+            }
+
+            // Enable Import button only if all paths are valid
+            databaseImportButton.SetEnabled(generatedPathValid && gameDataPathValid && databasePathValid);
         }
 
         /// <summary>

Packages/studio.shortsleeve.gamescriptunity/package.json

@@ -1,7 +1,7 @@
 {
   "name": "studio.shortsleeve.gamescriptunity",
   "displayName": "GameScriptUnity",
-  "version": "0.0.20",
+  "version": "0.0.21",
   "unity": "2023.2",
   "description": "Cross-platform dialogue middleware",
   "keywords": [