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

Correction UI du footer #728

Closed
5 tasks
ColineLebaratoux opened this issue May 6, 2024 · 7 comments · Fixed by #756
Closed
5 tasks

Correction UI du footer #728

ColineLebaratoux opened this issue May 6, 2024 · 7 comments · Fixed by #756
Assignees
Labels
dev front Issues liées à des problématiques de dev front minor fix Problème mineur simple à résoudre (éditorial, coquille, ...) valid Validé par l'équipe "non-tech"

Comments

@ColineLebaratoux
Copy link
Collaborator

ColineLebaratoux commented May 6, 2024

  • Remplacer le logo conseillers entreprises par service-public.fr

Image

  • Faire tenir l'ensemble des logos des partenaires sur une même ligne
  • Supprimer le logo TEE dans le footer
  • Ajouter une ligne de séparation entre les logos des partenaires et les liens vers les pages accessibilité,...
  • Revoir la version mobile

Actuel

Image

Version cible

Image
Image

@ColineLebaratoux ColineLebaratoux added minor fix Problème mineur simple à résoudre (éditorial, coquille, ...) dev front Issues liées à des problématiques de dev front labels May 6, 2024
@dolemoine
Copy link
Collaborator

dolemoine commented May 22, 2024

@ColineLebaratoux, quelques questions sur les points :

  • Remplacer le logo conseillers entreprises par service-public.fr : aujourd'hui le lien renvoie vers https://conseillers-entreprises.service-public.fr, je change pour https://www.service-public.fr ?
  • Supprimer le logo TEE dans le footer : le lien renvoie vers la page d'accueil TEE, qu'est-ce qu'on met à la place ?
  • comportement des logos partenaires : c'est le point de rupture qu'il faut probablement revoir, car sur une certaine largeur d'écran, on a bien l'allignement sur une ligne :

Image

Du coup je pense que ça va un peu de paire avec la révision de la version mobile, on peut peut-être se caler un point tous les deux pour ça ?

@ColineLebaratoux
Copy link
Collaborator Author

Hello @dolemoine !

  • Tu peux garder le lien vers https://conseillers-entreprises.service-public.fr/
  • Supprimer le logo TEE dans le footer => rien à ajouter à la place / on enlève le lien
  • est-ce que je peux te laisser faire une proposition pour le comportement des logos partenaires pour atteindre la cible des maquettes ? et on en discute après ?

@dolemoine
Copy link
Collaborator

@ColineLebaratoux voilà le lien pour le check de la version modifiée : https://tee-preprod-pr756.osc-fr1.scalingo.io/

@dolemoine dolemoine added the need validation Nécessite une validation par l'équipe label May 23, 2024
@dolemoine dolemoine linked a pull request May 23, 2024 that will close this issue
@ColineLebaratoux
Copy link
Collaborator Author

ColineLebaratoux commented May 24, 2024

Hello !
Top, ça fonctionne bien le passage entre les différentes taille d'écran
Je me demande juste pourquoi il n'y a pas le même espacement entre chaque logo ?

  • notamment entre service public et Ademe
  • et entre Ademe et l'icône "lien externe"
Capture d’écran 2024-05-24 à 09 37 33

@dolemoine
Copy link
Collaborator

@ColineLebaratoux j'ai corrigé l'espacement de l'icône "lien externe" et des logos partenaire.

Image

Pour le logo service public, il était de base plutôt alligné sur la gauche, comment souhaites-tu qu'il soit agencé ?
Tu peux me ping sur matermost si tu veux qu'on en parle en direct (c'est peut être plus simple ? )

@ColineLebaratoux
Copy link
Collaborator Author

C'est très bien, tu peux laisser comme ça !

@dolemoine
Copy link
Collaborator

Parfait,
Je te remets la lien pour validation : https://tee-preprod-pr756.osc-fr1.scalingo.io/

@ColineLebaratoux ColineLebaratoux added valid Validé par l'équipe "non-tech" and removed need validation Nécessite une validation par l'équipe labels May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev front Issues liées à des problématiques de dev front minor fix Problème mineur simple à résoudre (éditorial, coquille, ...) valid Validé par l'équipe "non-tech"
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants