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

[FO] Revue des menus, ajout de deux pages statiques, mise à jour DSFR 1.11.2 #2337

Merged
merged 10 commits into from
Mar 25, 2024

Conversation

emilschn
Copy link
Collaborator

@emilschn emilschn commented Mar 15, 2024

Ticket

#2321

Description

Le ticket lié comporte plusieurs sous-tâches qui sont gérées ailleurs (pages Contact et Statistiques).
Ce qui a été géré ici :

  • mise à jour des titres (<title>) des pages
  • mise à jour du menu pour reprendre le composant Navigation principale du DSFR
  • ajout d'un fil d'Ariane sur les pages front
  • ajustements de la page Qui sommes-nous ?
  • ajout de deux pages
    • Aides pour les travaux
    • Répartition des obligations d'entretien

Changements apportés

  • Mise à jour du DSFR en version 1.11 nécessaire pour utiliser des pictogrammes manquants sur les nouvelles pages
  • Ajout d'un nouveau fichier pdf dans les assets

Pré-requis

npm run build

Tests

  • Pour la MAJ du DSFR, il faudrait faire un tour complet du site

Tester ces pages en formats desktop et mobile, et vérifier correspondance avec maquettes.

  • Qui sommes-nous ?
  • Entretien du logement
  • Aides

Pour le fil d'Ariane

  • Contact
  • Mentions légales
  • CGU Usagers
  • CGU Agents
  • Politique de confidentialité
  • Accessibilité

@emilschn emilschn changed the title [FO] Revue des menus et ajout de deux pages [WIP] [FO] Revue des menus et ajout de deux pages Mar 15, 2024
@emilschn emilschn changed the title [WIP] [FO] Revue des menus et ajout de deux pages [FO] Revue des menus et ajout de deux pages Mar 18, 2024
@emilschn emilschn marked this pull request as ready for review March 18, 2024 14:30
@emilschn emilschn changed the title [FO] Revue des menus et ajout de deux pages [FO] Revue des menus et ajout de deux pages statiques Mar 18, 2024
@emilschn emilschn changed the title [FO] Revue des menus et ajout de deux pages statiques [FO] Revue des menus, ajout de deux pages statiques, mise à jour DSFR 1.11.2 Mar 18, 2024
Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

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

J'ai repéré deux soucis :

  • Les pages entretien du logement et aides au travaux n'active pas le menu à propos (voir commentaire de code)
  • En version mobile la page signalement perd son header/footer (seul le contenu reste visible)
    Screenshot 2024-03-20 at 15-05-39 Histologe

templates/header.html.twig Outdated Show resolved Hide resolved
@emilschn
Copy link
Collaborator Author

@numew
Bien vu pour le retour sur la page active dans le menu ! Je l'ai fait avant les pages et j'ai oublié de rajouter...

Pour la page signalement, c'est normal et c'est déjà le cas. C'est voulu par Mathilde. Tu peux le voir en prod :)
C'est un choix pour donner l'aspect un peu plus "appli". C'est discutable, mais volontaire.

Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

Profiter de cette PR pour revoir les liens du footer (les attributs title) ?

@emilschn
Copy link
Collaborator Author

@sfinx13 @numew ok !

Copy link

sonarcloud bot commented Mar 21, 2024

Quality Gate Passed Quality Gate passed

Issues
6 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

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

Ok

Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

Y'a une régression sur le lien gestion des cookie sur le BO ça doit ouvrir la modale de gestion de cookie sinon le reste est OK

image

@emilschn
Copy link
Collaborator Author

Y'a une régression sur le lien gestion des cookie sur le BO ça doit ouvrir la modale de gestion de cookie sinon le reste est OK

image

C'est toujours le cas :)
Le lien est différent entre front et bo, je n'ai rien changé ;)

Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

Lecture et test OK
J'avais oublié d'activer matomo

@sfinx13 sfinx13 merged commit 8c020ff into develop Mar 25, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the feature/2321-front-pages branch April 11, 2024 13:17
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.

None yet

3 participants