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

#2363 - fix retours sur la page de convention #2585

Merged
merged 9 commits into from
Nov 22, 2024

Conversation

celineung
Copy link
Contributor

@celineung celineung commented Nov 22, 2024

🌈 Description

Prise en compte des retours sur la page de convention:

  • ne plus avoir de chevauchements entre adresse mail et numéro de téléphone pour les résolutions entre 768 et 1200 px lorsqu'une adresse est trop longue
  • dans le bloc "Métier observé", remplacer "Objectif" par "Objectif de l'immersion", comme sur la maquette
  • passer le mot "SIRET" en lettres capitales
  • ajouter plus d'espace entre les lignes de semaines pour gagner en lisibilité
  • ajouter les flags "En attente de signature" pour les parties prenantes n'ayant pas encore signé
  • bloc "Information complémentaire" : ne mettre les ":" après le "Oui" que lorsque le champ de détail a été rempli
  • bloc "Convention" : Sur mobile le bouton passe sous l'illustration, il devrait être aligné avec l'ID de la convention
  • les textes sont en Corps de texte SM - 14px
  • les badges utilisés pour le statut de signature est le SM en font-size 12px

💯 Remarque

Pour la remarque:

Les adresses mail ne sont pas alignées en hauteur avec les valeurs des champs d'à côté (constaté dans tous les blocs)

Ceci est dû à l'icone qui permet de copier qui a une taille fixe définie par le DSFR.
Je verrai avec Enguerran dans un second temps ce que l'on peut faire.
Sinon, vu avec Gaël, ce n'est pas grave si pas possible de faire ce fix.

Copy link

@celineung celineung merged commit e3d26cf into main Nov 22, 2024
20 of 23 checks passed
@celineung celineung deleted the 2363-review-on-convention-page branch November 22, 2024 11:13
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.

2 participants