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

[FEATURE] Accompagner l'utilisateur mobile si son expérience risque d'être dégradée (PIX-13746) (PIX-13909) #9893

Merged
merged 7 commits into from
Aug 22, 2024

Conversation

yannbertrand
Copy link
Member

@yannbertrand yannbertrand commented Aug 21, 2024

🦄 Problème

En tant qu’utilisateur je souhaite pouvoir être informé si mon expérience sur mon module de formation va être dégradée afin de pouvoir changer de device si besoin.

🤖 Proposition

Dans les modules concernés, afficher une modale au lancement pour prévenir que l'expérience pourra ne pas être optimale.

Techniquement, on affiche la modale uniquement si la propriété tabletSupport du module n'est pas comfortable et en dessous du breakpoint tablette.

Les événements de tracking ont été mis à jour ainsi :

  • "Click sur le bouton Commencer" devient "Click sur le bouton Commencer un passage"
  • En petit écran sur les modules concernés :
    • Nouvel événement "Ouvre la modale d'alerte de largeur d'écran"
    • Nouvel événement "Ferme la modale d'alerte de largeur d'écran" peu importe la manière de fermer
    • Nouvel événement "Click sur le bouton Commencer un passage en petit écran"

On en profite pour réaligner les breakpoints utilisés en JS avec le code de Pix UI.

On s'est rendu compte que les boutons dans les modales n'étaient pas wrappés en mobile, on a profité d'une technique de @Jeyffrey pour mettre en commun ce comportement dans un composant ResponsiveUlWideWrap. On le réutilise également sur les éléments Download.

🌈 Remarques

On a utilisé la librairie ember-responsive pour pouvoir tester automatiquement les 2 usages.

💯 Pour tester

Sur un module concerné, par exemple sur bien-ecrire-son-adresse-mail :

  • La page de détails ne doit pas avoir changé, en "mobile/tablette" ou "desktop".

Sur un module concerné, par exemple le didacticiel :

  • En résolution "mobile/tablette", la nouvelle modale doit s'afficher.
  • En résolution "desktop", on accède au module comme avant.
  • Vérifier aussi que les boutons de l'élément "Download" reste accessible que ce soit en mobile/tablette ou desktop.

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@yannbertrand yannbertrand force-pushed the PIX-13909-mobile-alert branch 5 times, most recently from dc6fe6e to 5026163 Compare August 21, 2024 09:18
@yannbertrand yannbertrand marked this pull request as ready for review August 21, 2024 13:24
@yannbertrand yannbertrand requested a review from a team as a code owner August 21, 2024 13:24
@yannbertrand yannbertrand force-pushed the PIX-13909-mobile-alert branch 3 times, most recently from 34a06f6 to 857069b Compare August 21, 2024 13:29
@jujumathieu
Copy link

Quand j'affiche la fenêtre à un format de smartphone/tablette "connu", la modale s'affiche bien (et la navigation clavier fonctionne bien). En revanche, si je choisi le mode "responsive" de l'inspecteur chrome, le bouton "revenir aux détails" mord sur la gauche de l'écran à partir du moment où la largeur de l'écran fait moins de 360px..

Capture d’écran 2024-08-22 à 10 43 34

@yannbertrand
Copy link
Member Author

Quand j'affiche la fenêtre à un format de smartphone/tablette "connu", la modale s'affiche bien (et la navigation clavier fonctionne bien). En revanche, si je choisi le mode "responsive" de l'inspecteur chrome, le bouton "revenir aux détails" mord sur la gauche de l'écran à partir du moment où la largeur de l'écran fait moins de 360px..

C'est corrigé, merci pour l'alerte ! On l'a également mis en place sur les éléments "Download" pour que les boutons de téléchargement aient le même comportement en mobile.

@jujumathieu jujumathieu added the Func Review OK PO validated functionally the PR label Aug 22, 2024
@jujumathieu
Copy link

C'est bon pour la responsivité des boutons de téléchargement de doc et de la modale. Bien joué !

@pix-service-auto-merge pix-service-auto-merge merged commit 2f6b4e5 into dev Aug 22, 2024
7 checks passed
@yannbertrand yannbertrand changed the title [FEATURE] Accompagner l'utilisateur mobile si son expérience risque d'être dégradée (PIX-13746) [FEATURE] Accompagner l'utilisateur mobile si son expérience risque d'être dégradée (PIX-13746) (PIX-13909) Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants