Skip to content

MAJ du DSFR en 1.10.2#576

Merged
bellangerq merged 6 commits intomainfrom
119-mettre-à-jour-le-dsfr-déplacement-curseur-au-clavier-fonctionne-pas
Dec 6, 2023

Hidden character warning

The head ref may contain hidden characters: "119-mettre-\u00e0-jour-le-dsfr-d\u00e9placement-curseur-au-clavier-fonctionne-pas"
Merged

MAJ du DSFR en 1.10.2#576
bellangerq merged 6 commits intomainfrom
119-mettre-à-jour-le-dsfr-déplacement-curseur-au-clavier-fonctionne-pas

Conversation

@bellangerq
Copy link
Copy Markdown
Collaborator

@bellangerq bellangerq commented Nov 30, 2023

Quelques éléments ont changé visuellement et qui nécessitent une revue / un changement :

  • fr-accordion : couleur de texte du titre + état ouvert
  • fr-tile : disparait et est remplacé. L'usage des tiles sur la home et l'étap 1 ne semble plus correspondre au composant (car pas de lien sur la home ni sur l'étape 1). Du coup : est-ce qu'on remplace par un composant déjà fourni ou on fait du custom ?

J'ai fait le tour de l'app mais il est possible que j'ai oublié des éléments qui ont changé visuellement.


closes #119

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 30, 2023

Deploy Preview for confiture ready!

Name Link
🔨 Latest commit 122cf7c
🔍 Latest deploy log https://app.netlify.com/sites/confiture/deploys/6569ec0db4d26e0008cb4776
😎 Deploy Preview https://deploy-preview-576--confiture.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@benoitdequick
Copy link
Copy Markdown
Contributor

y'a pas de preview ?

@bellangerq
Copy link
Copy Markdown
Collaborator Author

bellangerq commented Nov 30, 2023

y'a pas de preview ?

La version de Node actuelle (16) est pas supportée par le DSFR, il faut passer à la 18. Je suis dessus !

@AdrienMuzyczka
Copy link
Copy Markdown
Collaborator

AdrienMuzyczka commented Nov 30, 2023

Si on veut être conforme aux recos du DSFR il faudrait remplacer le composant par un simple "bloc" avec contour gris (Comme les blocs "pages" dans "'échantillon des pages à auditer" de l'étape 1)

On retrouve aussi ces tuiles dans le page de synthèse.

@bellangerq
Copy link
Copy Markdown
Collaborator Author

C'est bon pour la preview @benoitdequick.

Du coup il reste plus que l'histoire des fr-accordion, les tiles ont été supprimées et remplacées par des simples cadres.

@bellangerq bellangerq marked this pull request as ready for review December 1, 2023 10:21
@hissalht
Copy link
Copy Markdown
Collaborator

hissalht commented Dec 1, 2023

Pour la version de node, il faut aussi la mettre à jour la version de node utilisée pour build le container docker du front dans confiture-web-app/Dockerfile. Juste changer le node:16 en node:18 sur la première ligne.

@AdrienMuzyczka
Copy link
Copy Markdown
Collaborator

@bellangerq, ok pour garder les modifications apportées sur les accordéons par la V1.10 (passage en bleu des libellés + ajout d'états sur le composant). Les éléments interactifs sont mis en évidence.

@bellangerq bellangerq merged commit 473fae7 into main Dec 6, 2023
@bellangerq bellangerq deleted the 119-mettre-à-jour-le-dsfr-déplacement-curseur-au-clavier-fonctionne-pas branch December 6, 2023 13:15
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.

Mettre à jour le DSFR. Déplacement curseur au clavier fonctionne pas

4 participants