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

Corrige l'initialisation du menu mobile sur tablette #6220

Merged
merged 1 commit into from
Jan 8, 2022

Conversation

Luuka
Copy link
Contributor

@Luuka Luuka commented Jan 4, 2022

Petite PR pour faire en sorte que le menu mobile soit initialisé sur les écrans entre 960 et 1024px.
Sur ces écrans le bouton "burger" s'affiche mais le menu ne s'ouvre pas.

C'est le cas lorsque j'affiche ZDS sur un demi écran 21/9.

⚠️ Le bug ne se produit pas lorsqu'on redimensionne d'une résolution supérieure 960 jusqu'à une résolution entre 960 et 1024 car dans ce cas le menu est déjà initialisé.

Contrôle qualité

  • Se rendre sur une page de ZDS affichant le menu
  • Redimensionner sa fenêtre pour atteindre une largeur entre 960 et 1024px
  • Recharger la page
  • Le bouton "burger" fonctionne et ouvre le menu principal

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.019% when pulling a3bad05 on Luuka:fix-mobile-menu-tablet into 915c7d1 on zestedesavoir:dev.

@Situphen
Copy link
Member

Situphen commented Jan 4, 2022

Ton bug se produit avec quel navigateur ?

Avec Firefox 95 et Google Chrome 96, je peux ouvrir le menu latéral entre ± 960 px et ± 1024 px, par contre il ne se ferme pas donc il y a bien un soucis !

@Luuka
Copy link
Contributor Author

Luuka commented Jan 4, 2022

Je reproduis sur Firefox (95) et Chrome (96) à la fois sur MacOS et Windows

Par contre je reproduis ton comportement quand je redimensionne depuis une résolution < 960px vers une résolution supérieure à 960px. Dans ce cas l'ouverture du menu fonctionne mais pas sa fermeture.

Mon correctif corrige les deux problèmes d'après mes tests

Copy link
Member

@Situphen Situphen left a comment

Choose a reason for hiding this comment

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

QA OK Le petit soucis que j'ai pu reproduire a disparu et tout fonctionne correctement, merci !

@Situphen Situphen merged commit 2ba9222 into zestedesavoir:dev Jan 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants