Hide dropdown menu instead of removing it from DOM (#881)

* Hide dropdown menu instead of removing it from DOM Note: no more smooth transition after this change * Update CHANGELOG
DISIC · Dec 18, 2024 · 9a901bd · 9a901bd
1 parent 0427bae
commit 9a901bd
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 13 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,7 @@ Tous les changements notables de Ara sont documentés ici avec leur date, leur c
 
 ### Corrections 🐛
 
+- Corrige les menus déroulants en rendant le code HTML généré valide ([#881](https://github.com/DISIC/Ara/pull/881))
 - Corrige la mauvaise position verticale dans la page après utilisation des ancres « Pages » du rapport ([#879](https://github.com/DISIC/Ara/pull/879))
 
 ## 13/12/2024

diff --git a/confiture-web-app/src/components/ui/Dropdown.vue b/confiture-web-app/src/components/ui/Dropdown.vue
@@ -85,19 +85,17 @@ defineExpose({ buttonRef, closeOptions });
     >
       {{ title }}
     </button>
-    <Transition>
-      <div
-        v-if="showContent"
-        :id="`dropdown-${uniqueId}`"
-        :class="[
-          'fr-p-2w dropdown-content',
-          { 'dropdown-content-left': alignLeft }
-        ]"
-        role="menu"
-      >
-        <slot />
-      </div>
-    </Transition>
+    <div
+      :id="`dropdown-${uniqueId}`"
+      :hidden="!showContent"
+      :class="[
+        'fr-p-2w dropdown-content',
+        { 'dropdown-content-left': alignLeft }
+      ]"
+      role="menu"
+    >
+      <slot />
+    </div>
   </div>
 </template>