add the required indicator to the label

beautyjoy · Aug 28, 2024 · 69e5d98 · 69e5d98
1 parent fcb58d0
commit 69e5d98
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/app/javascript/packs/schools.js b/app/javascript/packs/schools.js
@@ -20,10 +20,12 @@ $("#new_school").on("submit", function (e) {
 });
 
 // TODO: This list should match the model attributes.
-const SCHOOL_REQUIRED_ATTRIBUTES = ['name', 'city', 'state', 'website', 'grade_level', 'school_type'];
+// These are all prefixed with schol_ on the page form.
+const SCHOOL_REQUIRED_ATTRIBUTES = ['name', 'city', 'state', 'website', 'grade_level', 'type'];
 
-let toggle_required = (selectors, state) => {
-  selectors.forEach(s => $(`#school_${s}`).prop("required", state))
+let toggle_required = (fields, state) => {
+  fields.forEach(s => $(`#school_${s}`).prop("required", state));
+  fields.forEach(s => $(`[for=school_${s}]`).addClass('label-required'));
 }
 
 let create_school = function (input, callback) {