Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hide dropdown menu instead of removing it from DOM #881

Conversation

yaaax
Copy link
Collaborator

@yaaax yaaax commented Nov 15, 2024

Note: no more smooth transition after this change

@yaaax yaaax requested review from hissalht and bellangerq November 15, 2024 16:32
@yaaax yaaax added the RGAA label Nov 15, 2024
@hissalht hissalht temporarily deployed to ara-880-code-html-non-v-siaj8s November 15, 2024 16:36 Inactive
@bellangerq
Copy link
Collaborator

Note: no more smooth transition after this change

Perso pas de problème avec ça.

:id="`dropdown-${uniqueId}`"
:hidden="!showContent"
:class="[
{ 'fr-hidden': !showContent },
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Besoin de doubler l'attribut hidden avec une classe ?

Copy link
Collaborator Author

@yaaax yaaax Dec 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En effet, inutile ici, j’ai supprimé ;)

Merci !

@yaaax yaaax force-pushed the 880-code-html-non-valide-avant-ouverture-le-menu-télécharger-est-référencé-par-son-bouton-mais-nest-pas-présent-dans-le-dom branch from 58bdf55 to ed437b3 Compare December 18, 2024 11:57
@hissalht hissalht temporarily deployed to ara-880-code-html-non-v-l7hp0u December 18, 2024 13:47 Inactive
@yaaax yaaax force-pushed the 880-code-html-non-valide-avant-ouverture-le-menu-télécharger-est-référencé-par-son-bouton-mais-nest-pas-présent-dans-le-dom branch from ed437b3 to 856267f Compare December 18, 2024 17:27
@hissalht hissalht temporarily deployed to ara-880-code-html-non-v-l7hp0u December 18, 2024 17:28 Inactive
Note: no more smooth transition after this change
@yaaax yaaax force-pushed the 880-code-html-non-valide-avant-ouverture-le-menu-télécharger-est-référencé-par-son-bouton-mais-nest-pas-présent-dans-le-dom branch from 856267f to 89dcdd2 Compare December 18, 2024 17:29
@hissalht hissalht temporarily deployed to ara-880-code-html-non-v-l7hp0u December 18, 2024 17:36 Inactive
@yaaax yaaax merged commit 9a901bd into main Dec 18, 2024
1 check passed
@yaaax yaaax deleted the 880-code-html-non-valide-avant-ouverture-le-menu-télécharger-est-référencé-par-son-bouton-mais-nest-pas-présent-dans-le-dom branch December 18, 2024 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants