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

[BUGFIX] Le scroll auto ne descend pas assez bas (PIX-15476) #10675

Merged
merged 2 commits into from
Dec 4, 2024

Conversation

dianeCdrPix
Copy link
Contributor

🍂 Problème

La barre de navigation n’est pas prise en compte dans le calcul du scroll, il faudrait inclure sa hauteur pour éviter de tomber au milieu d’un texte

🌰 Proposition

Utiliser un document.getelementById pour récupérer l'élément Navbar, le passer à la fonction focusAndScroll() et calculer la hauteur à laquelle afficher l'écran en fonction de l'épaisseur de la Navbar.

🎃 Remarques

  • Solution alternative suite à la PR créée sur le même sujet ici
  • Dans la fonction focusAndScroll, on a dû ajouter en paramètre la fonction getNavbar qui permet de récupérer l'élement Navbar sur la page du module. Cela permet notamment d'avoir un test unitaire fonctionnel

🪵 Pour tester

  • Ouvrir un module
  • Cliquer sur "Continuer" ou "Passer" sur des grains et voir comment se comporte le scroll
  • Zoomer la page sur le navigateur
  • Cliquer sur "Continuer" ou "Passer" sur des grains et voir comment se comporte le scroll

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

Copy link
Member

@yannbertrand yannbertrand left a comment

Choose a reason for hiding this comment

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

OK avec le concept 👍

@yannbertrand yannbertrand changed the title [FEATURE] Solution Alternative - Le scroll auto ne descend pas assez bas (PIX-15476) [BUGFIX] Le scroll auto ne descend pas assez bas (PIX-15476) Dec 2, 2024
@clemlatz clemlatz force-pushed the PIX-15476-alternative-solution branch 2 times, most recently from 2924c74 to 1cc4632 Compare December 2, 2024 16:21
@yannbertrand yannbertrand force-pushed the PIX-15476-alternative-solution branch from 1cc4632 to 02f3ed3 Compare December 3, 2024 13:24
@dianeCdrPix dianeCdrPix force-pushed the PIX-15476-alternative-solution branch from 02f3ed3 to 0fe58d4 Compare December 4, 2024 09:02
@yannbertrand yannbertrand force-pushed the PIX-15476-alternative-solution branch from 0fe58d4 to 5bcd53f Compare December 4, 2024 09:45
@pix-service-auto-merge pix-service-auto-merge force-pushed the PIX-15476-alternative-solution branch from 5bcd53f to a71644a Compare December 4, 2024 09:51
@pix-service-auto-merge pix-service-auto-merge force-pushed the PIX-15476-alternative-solution branch from a71644a to a669b78 Compare December 4, 2024 10:25
@pix-service-auto-merge pix-service-auto-merge merged commit bc489d0 into dev Dec 4, 2024
7 of 9 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the PIX-15476-alternative-solution branch December 4, 2024 10:31
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.

5 participants