From c1ae5bfd301121fa70e7aadc2d0e081b77563e64 Mon Sep 17 00:00:00 2001
From: nico <contact@nardu.in>
Date: Thu, 20 Feb 2025 17:16:46 +0100
Subject: [PATCH] fix(a11y forms): dataset availability

---
 .../forms/dataset/DatasetPropertiesFields.vue | 133 +++++++++---------
 .../forms/dataset/SelectDataset.vue           |   6 +
 src/utils/form.ts                             |   2 +-
 3 files changed, 76 insertions(+), 65 deletions(-)

diff --git a/src/components/forms/dataset/DatasetPropertiesFields.vue b/src/components/forms/dataset/DatasetPropertiesFields.vue
index 67e44946..821e5e36 100644
--- a/src/components/forms/dataset/DatasetPropertiesFields.vue
+++ b/src/components/forms/dataset/DatasetPropertiesFields.vue
@@ -136,73 +136,74 @@ onMounted(() => {
     :error-title="getErrorMessage('title')"
     :error-purpose="getErrorMessage('purpose')"
   />
-  <div class="fr-mt-1w fr-mb-4w">
-    <SelectDataset
-      v-model="selectedDataset"
-      :already-selected-datasets="alreadySelectedDatasets"
-      @update:model-value="onSelectDataset"
-    />
-  </div>
-  <div
-    v-if="!selectedDataset && topicsDatasetEditorialization"
-    class="fr-mt-4w"
-  >
-    <fieldset
-      id="input-availability"
-      tabindex="-1"
-      class="fr-fieldset"
-      role="radiogroup"
-      aria-errormessage="errors-availability"
-      :aria-invalid="formErrors.includes('availability') ? true : undefined"
-    >
-      <legend class="fr-fieldset__legend fr-fieldset__legend--regular">
-        Vous ne trouvez pas le jeu de données dans data.gouv.fr&nbsp;?
-      </legend>
-      <DsfrRadioButton
-        v-model="datasetProperties.availability"
-        name="source"
-        :value="Availability.URL_AVAILABLE"
-        label="J'ajoute l'URL"
+  <div id="input-availability" tabindex="-1">
+    <div class="fr-mt-1w fr-mb-4w">
+      <SelectDataset
+        v-model="selectedDataset"
+        :already-selected-datasets="alreadySelectedDatasets"
+        :is-invalid="formErrors.includes('availability')"
+        @update:model-value="onSelectDataset"
       />
-      <div
-        v-if="datasetProperties.availability === Availability.URL_AVAILABLE"
-        class="fr-mb-4w fr-fieldset__element"
+    </div>
+    <div
+      v-if="!selectedDataset && topicsDatasetEditorialization"
+      class="fr-mt-4w"
+    >
+      <fieldset
+        class="fr-fieldset availability"
+        role="radiogroup"
+        aria-errormessage="errors-availability"
+        :aria-invalid="formErrors.includes('availability') ? true : undefined"
       >
-        <DsfrInput
-          id="input-availabilityUrl"
-          v-model="datasetProperties.uri"
-          label="Url vers le jeu de données souhaité (obligatoire)"
-          :label-visible="true"
-          class="fr-mb-md-1w fr-input"
-          aria-errormessage="errors-availabilityUrl"
-          :aria-invalid="
-            formErrors.includes('availabilityUrl') ? true : undefined
-          "
+        <legend class="fr-fieldset__legend fr-fieldset__legend--regular">
+          Vous ne trouvez pas le jeu de données dans data.gouv.fr&nbsp;?
+        </legend>
+        <DsfrRadioButton
+          v-model="datasetProperties.availability"
+          name="source"
+          :value="Availability.URL_AVAILABLE"
+          label="J'ajoute l'URL"
         />
-        <ErrorMessage
-          v-if="!!getErrorMessage('availabilityUrl')"
-          input-name="availabilityUrl"
-          :error-message="getErrorMessage('availabilityUrl')"
+        <div
+          v-if="datasetProperties.availability === Availability.URL_AVAILABLE"
+          class="fr-mb-4w fr-fieldset__element"
+        >
+          <DsfrInput
+            id="input-availabilityUrl"
+            v-model="datasetProperties.uri"
+            label="Url vers le jeu de données souhaité (obligatoire)"
+            :label-visible="true"
+            class="fr-mb-md-1w fr-input"
+            aria-errormessage="errors-availabilityUrl"
+            :aria-invalid="
+              formErrors.includes('availabilityUrl') ? true : undefined
+            "
+          />
+          <ErrorMessage
+            v-if="!!getErrorMessage('availabilityUrl')"
+            input-name="availabilityUrl"
+            :error-message="getErrorMessage('availabilityUrl')"
+          />
+        </div>
+        <DsfrRadioButton
+          v-model="datasetProperties.availability"
+          name="source"
+          :value="Availability.MISSING"
+          label="Je n'ai pas trouvé la donnée"
         />
-      </div>
-      <DsfrRadioButton
-        v-model="datasetProperties.availability"
-        name="source"
-        :value="Availability.MISSING"
-        label="Je n'ai pas trouvé la donnée"
-      />
-      <DsfrRadioButton
-        v-model="datasetProperties.availability"
-        name="source"
-        :value="Availability.NOT_AVAILABLE"
-        label="Je n'ai pas cherché la donnée"
-      />
-      <ErrorMessage
-        v-if="!!getErrorMessage('availability')"
-        :error-message="getErrorMessage('availability')"
-        input-name="availability"
-      />
-    </fieldset>
+        <DsfrRadioButton
+          v-model="datasetProperties.availability"
+          name="source"
+          :value="Availability.NOT_AVAILABLE"
+          label="Je n'ai pas cherché la donnée"
+        />
+      </fieldset>
+    </div>
+    <ErrorMessage
+      v-if="!!getErrorMessage('availability')"
+      :error-message="getErrorMessage('availability')"
+      input-name="availability"
+    />
   </div>
   <div class="fr-input-group">
     <SelectTopicGroup
@@ -219,7 +220,11 @@ onMounted(() => {
 .fr-fieldset {
   margin: 30px 0;
 }
-.fr-fieldset:focus {
+.fr-fieldset.availability[aria-invalid='true'] {
+  border: none;
+  outline: 2px solid var(--border-plain-error);
+}
+#input-availability:focus {
   outline-style: solid;
 }
 fieldset legend {
diff --git a/src/components/forms/dataset/SelectDataset.vue b/src/components/forms/dataset/SelectDataset.vue
index 34ec5d8f..bcd261c9 100644
--- a/src/components/forms/dataset/SelectDataset.vue
+++ b/src/components/forms/dataset/SelectDataset.vue
@@ -20,6 +20,10 @@ const props = defineProps({
   alreadySelectedDatasets: {
     type: Array<DatasetProperties>,
     default: []
+  },
+  isInvalid: {
+    type: Boolean,
+    default: false
   }
 })
 
@@ -85,6 +89,8 @@ const clear = () => {
     no-options-text="Aucun jeu de données trouvé, précisez ou élargissez votre recherche."
     :aria="{
       'aria-describedby': 'dataset-description',
+      'aria-errormessage': 'errors-availability',
+      'aria-invalid': `${isInvalid ? true : undefined}`,
       // useless or unsupported https://github.com/vueform/multiselect/issues/436
       'aria-labelledby': null,
       'aria-multiselectable': null,
diff --git a/src/utils/form.ts b/src/utils/form.ts
index c46234a6..ba127a8a 100644
--- a/src/utils/form.ts
+++ b/src/utils/form.ts
@@ -27,7 +27,7 @@ const errorMessages = [
   },
   {
     inputName: 'availability',
-    message: 'La disponibilité doit être renseignée.'
+    message: 'Un jeu de données ou une disponibilité doit être sélectionné.'
   },
   {
     inputName: 'availabilityUrl',