making saves-settings.hbs template more generic.

fixed required attribute bug for when it is passed into the template as false (it was correct for values of true). orffen#69
DC23 · Nov 27, 2024 · fc58096 · fc58096
1 parent 343027a
commit fc58096
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 9 deletions.
diff --git a/module/settings/saves-settings.mjs b/module/settings/saves-settings.mjs
@@ -40,7 +40,7 @@ class SavesSettings extends FormApplication {
     return foundry.utils.mergeObject(super.defaultOptions, {
       popOut: true,
       width: 400,
-      template: `systems/${SYSTEM_ID}/templates/settings/saves-settings.hbs`,
+      template: `systems/${SYSTEM_ID}/templates/settings/string-array-settings.hbs`,
       id: SETTINGS.SAVES_MENU,
       title: 'BASICFANTASYRPG.Settings.SavesMenu.name',
     })
@@ -55,6 +55,7 @@ class SavesSettings extends FormApplication {
         id: v,
         label: SavesSettings.defaultSaves[v],
         value: initialValues[v],
+        required: true,
       }
     })
     return data

diff --git a/templates/settings/saves-settings.hbs → templates/settings/string-array-settings.hbs b/templates/settings/saves-settings.hbs → templates/settings/string-array-settings.hbs
@@ -1,17 +1,18 @@
-{{#*inline "savePartial"}}
+{{!-- fa
+ --}}
+
+{{#*inline "stringFieldPartial"}}
 <div class="form-row flexrow">
     <label for="{{id}}">{{label}}:</label>
-    <input name="{{id}}" required=true value="{{value}}" />
+    <input name="{{id}}" {{#if required}}required="true"{{/if}} value="{{value}}" />
 </div>
 {{/inline}}
 
+{{log this}}
 <form class="form-group flexcol">
-    <fieldset>
-        <legend>{{localize "BASICFANTASYRPG.Settings.SavesMenu.name"}}:</legend>
-        {{#each this}}
-        {{> savePartial}}
-        {{/each}}
-    </fieldset>
+    {{#each this}}
+    {{> stringFieldPartial}}
+    {{/each}}
 
     <footer class="sheet-footer flexrow button-container">
         <button class="reset-all" type="button" name="reset" data-action="reset">