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

Gestion du scroll #1204

Closed
Clemog opened this issue Jul 11, 2023 · 1 comment · Fixed by #1254
Closed

Gestion du scroll #1204

Clemog opened this issue Jul 11, 2023 · 1 comment · Fixed by #1254
Assignees
Labels
bug Something isn't working tech UX

Comments

@Clemog
Copy link
Contributor

Clemog commented Jul 11, 2023

Dans un site classique à l'ancienne, chaque lien ouvre une nouvelle page, donc le navigateur repart du haut.

NGC est une SPA : lorsque l'on change de page, le scroll "ne remonte pas" en haut de page.

(Exemple en cliquant sur les liens en bas de : https://nosgestesclimat.fr/guide ou dans les liens du plan du site.)

L'idée serait d'utiliser https://reactrouter.com/en/main/components/scroll-restoration pour gérer ce scroll afin qu'il remonte au moins pour les pages que l'on cible

Ça semble être assez rapide à faire

@Clemog Clemog added bug Something isn't working tech UX labels Jul 11, 2023
@Clemog Clemog mentioned this issue Jul 11, 2023
18 tasks
@dxb
Copy link
Collaborator

dxb commented Jul 25, 2023

Il y a un souci avec scroll-restoration, il nécéssite d'utiliser un router compatible avec l'api data :
https://reactrouter.com/en/6.14.2/routers/picking-a-router

ce qui nécessite de modifier le système de route.

D'un autre côté, on a un composant scrollToTop dans le base de code qui gère ça... comme sur la documentation.
Je peux l'ajouter sur le guide et les différentes pages qui n'en sont pas équipées.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tech UX
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants