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

Mise à jour du header pour synchroniser avec la nouvelle version DSFR #1201

Closed
lucaa opened this issue Apr 21, 2023 · 4 comments
Closed

Mise à jour du header pour synchroniser avec la nouvelle version DSFR #1201

lucaa opened this issue Apr 21, 2023 · 4 comments
Assignees
Labels
UX / UI All issues related to the presentation of the data
Milestone

Comments

@lucaa
Copy link
Collaborator

lucaa commented Apr 21, 2023

image

à changer:

  • nom du site à gauche: taille et gras de la police
  • survol marqué en gris
  • menu à droite:
    • titre du lien vers l'observatoire
    • gras du menu
    • plus de séparateur
    • padding au survol
@lucaa lucaa added the UX / UI All issues related to the presentation of the data label Apr 21, 2023
@lucaa lucaa added this to the 5.2 milestone Apr 21, 2023
@lucaa lucaa self-assigned this Apr 21, 2023
@lucaa lucaa modified the milestones: 5.2, 5.1.1 Apr 24, 2023
@ClementNumericite
Copy link

Je ne sais plus si on avait statué sur les icônes une fois connecté @lucaa
Capture d’écran du 2023-04-24 18-43-13

Il me semble qu'on avait évoqué la possibilité de les supprimer, mais nous n'en avons pas reparlé au dernier point.

@lucaa
Copy link
Collaborator Author

lucaa commented Apr 24, 2023

non, on n'en a pas parlé.

De ce que je comprends maintenant, il n'y aura pas de synchronisation visuelle à faire de ce point de vue.
Ainsi, un changement à ce niveau sera un changement de design pur, sans objectif de synchronisation. Comme son objectif est légèrement différent (et sa priorité aussi), peut-on capturer ce changement (s'il est nécessaire), dans un ticket séparé?

@clemdee
Copy link
Collaborator

clemdee commented May 3, 2023

Les changements possibles ont été implémentés.

Par contre, le gras du menu à droite ne peut pas être implémenté, car c'est un gras moyen (500) correspondant à une police du nouveau dsfr que nous n'avons pas sur notre ancienne version. Notre niveau de gras supporté est un niveau 700, qui apparait donc beaucoup plus gras que ce qui est attendu. Pour l'instant j'ai implémenté le gras en font-weight: 500, qui du coup n'est pas implémenté et le texte s'affiche donc normalement.

@clemdee clemdee closed this as completed May 3, 2023
@lucaa
Copy link
Collaborator Author

lucaa commented May 4, 2023

Je ne sais plus si on avait statué sur les icônes une fois connecté @lucaa Capture d’écran du 2023-04-24 18-43-13

Il me semble qu'on avait évoqué la possibilité de les supprimer, mais nous n'en avons pas reparlé au dernier point.

Après analyse, il semble que les menus de notifications et recherche sont affichés même pour les utilisateurs simples (non-admins).

J'ai crée les tickets #1206 et #1207 pour supprimer ces 2 menus.

Les notifications (#1206 ) sont probablement pas utilisés mais la recherche (#1207) est bien fonctionnelle et pourrait être utilisée par certains utilisateurs. L'accès à cette fonctionnalité restera disponible sur la page d'accueil.

Le menu burger, quand à lui, est affiché uniquement aux administrateurs et est nécessaire pour accéder aux fonctionnalités d'administration.

clemdee pushed a commit that referenced this issue May 5, 2023
…n DSFR

* Mise à jour des styles active/focus du logo du header
clemdee pushed a commit that referenced this issue May 9, 2023
…n DSFR

* Fix: Logo hover background overflowing on mobile layout
@github-project-automation github-project-automation bot moved this to Done: Closed & Delivered in Développement Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UX / UI All issues related to the presentation of the data
Projects
Status: Done: Closed & Delivered
Development

No branches or pull requests

3 participants