35522 add group edit page

AAFC-BICoE · Jan 16, 2025 · 4d878b8 · 4d878b8
1 parent c9ce929
commit 4d878b8
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/packages/dina-ui/components/group/GroupLabelsEditor.tsx b/packages/dina-ui/components/group/GroupLabelsEditor.tsx
@@ -78,14 +78,16 @@ export function GroupLabelsEditor({ valuesPath }: GroupLabelsEditorProps) {
                       <FormikButton
                         className="btn remove-attribute"
                         onClick={(_, form) => {
-                          // Delete the value and hide the managed attribute:
+                          // Delete the value and hide the label:
                           form.setFieldValue(
                             `${valuesPath}.${language}`,
                             undefined
                           );
+                          // Add the language back to availableLanguages
                           setAvailableLanguages(
                             concat(availableLanguages, language).sort()
                           );
+                          // Remove the language from visibleLanguages
                           setVisibleLanguages((languages) =>
                             languages.filter((lang) => lang !== language)
                           );
@@ -104,12 +106,15 @@ export function GroupLabelsEditor({ valuesPath }: GroupLabelsEditorProps) {
                   onChange={(value) => {
                     const selectedLang = value?.value;
                     if (selectedLang) {
+                      // add the language to visibleLanguages
                       setVisibleLanguages(
                         concat(visibleLanguages, selectedLang)
                       );
+                      // remove the language from availableLanguages
                       setAvailableLanguages((languages) =>
                         languages.filter((lang) => lang !== selectedLang)
                       );
+                      // set fieldValue in the formik.
                       form.setFieldValue(`${valuesPath}.${selectedLang}`, "");
                     }
                   }}