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

Pix 15834 display popup for smarphones and vertical tablets #11041

Conversation

rouxxi
Copy link
Contributor

@rouxxi rouxxi commented Jan 9, 2025

🎄 Problème

Certains élèves utilisent leur tablette en orientation portrait alors que l'affichage de Pix Junior est optimisé pour une orientation paysage.
Pix Junior est également accessible avec smartphone alors que l'application n'est pas utilisable avec ce type d'écran.

🎁 Proposition

Afficher une popup pour d'avertissement indiquant que l'affichage optimal est en paysage lorsque l'élève utilise Pix Junior en portrait sur une tablette.
Afficher une page fixe indiquant que Pix Junior n'est pas destiné aux smartphones.

🧦 Remarques

Pour que la popup ne s'affiche pas constamment, celle-ci ne s'affiche qu'une fois par session d'un même élève.

🎅 Pour tester

Accéder à Pix Junior avec un smartphone -> une page fixe indique que Pix Junior n'est pas destiné aux smartphones. Un lien vers la page Pix site dédiée est présent dans la page.

Accéder à Pix Junior avec une tablette en mode portrait -> une popup s'affiche.

  • Elle peut être fermée et elle n'est pas ré-affichée tout au long de la session de l'utilisateur.
  • Elle disparait si l'on tourne la tablette et elle n'est pas ré-affichée tout au long de la session de l'utilisateur. Quand un nouvel utilisateur s'identifie et positionne sa tablette en portrait, la popup se ré-affiche.

@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 :

@rouxxi rouxxi force-pushed the pix-15834-display-popup-for-smarphones-and-vertical-tablets branch from a010d86 to 5f3d457 Compare January 9, 2025 16:36
@mcampourcy mcampourcy force-pushed the pix-15834-display-popup-for-smarphones-and-vertical-tablets branch 3 times, most recently from 70ab6f2 to ec2563d Compare January 10, 2025 09:13
@aurelie-crouillebois aurelie-crouillebois added Func Review OK PO validated functionally the PR and removed Development in progress labels Jan 10, 2025
@aurelie-crouillebois aurelie-crouillebois force-pushed the pix-15834-display-popup-for-smarphones-and-vertical-tablets branch from ec2563d to b71fde2 Compare January 10, 2025 09:49
@aurelie-crouillebois aurelie-crouillebois force-pushed the pix-15834-display-popup-for-smarphones-and-vertical-tablets branch from b71fde2 to d81bd2d Compare January 10, 2025 09:57
@pierrepougetpix
Copy link

PR vallidée niveau design mais avec un bug sur iPhone où l'illustration du robot ne s'affiche pas

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15834-display-popup-for-smarphones-and-vertical-tablets branch from d81bd2d to b6f6a9d Compare January 16, 2025 09:53
@pix-service-auto-merge pix-service-auto-merge merged commit 505597e into dev Jan 16, 2025
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15834-display-popup-for-smarphones-and-vertical-tablets branch January 16, 2025 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants