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

Arriver en haut de la page quand on revient sur l’évènement après création/modif fiche zone #744

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

alanzirek
Copy link
Collaborator

@alanzirek alanzirek commented Feb 21, 2025

Cette PR permet d'arriver en haut de la page quand on revient sur l’évènement après création ou modification d'une fiche zone délimitée.

Initialement, le lien de redirection incluait un hash (#tabpanel-zone-panel) vers l'onglet zone et au chargement de la page la navigateur se positionné dessus. Par conséquent, le haut de la page de l'évènement n'était plus visible.

Ce qui a été fait :

  • suppression du hash pour supprimer l'effet de positionnement natif du navigateur sur l'onglet Zone,
  • ajout du paramètre tab dans l'URL suite à une création ou modification d'une fiche zone délimitée,
  • au chargement de la page, si le paramètre tab est a zone, alors sélection automatique de l'onglet (via API js DSFR)
  • ajout d'un écouteur dsfr.ready ou dsfr.start pour éviter que la fonction ne s’exécute avant le chargement du DSFR (si non présent, erreur js possible -> Uncaught TypeError: dsfr(...) is null)
  • utilisation d'un settimeout pour s'assurer que le DSFR soit chargé (à remplacer par dsfr.ready ou dsfr.start lors du merge de MAJ DSFR v1.13.0 #743)

@alanzirek alanzirek changed the title Arriver en haut de la page quand on revient sur l’évènement Arriver en haut de la page quand on revient sur l’évènement après création/modif fiche zone Feb 21, 2025
@alanzirek alanzirek force-pushed the scroll-top-after-fzd-form branch from 4faa431 to 894071b Compare February 26, 2025 08:58
@alanzirek alanzirek marked this pull request as ready for review February 26, 2025 09:16
@alanzirek alanzirek requested a review from Anto59290 February 26, 2025 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant