-
Notifications
You must be signed in to change notification settings - Fork 4
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
Comments
Je ne sais plus si on avait statué sur les icônes une fois connecté @lucaa Il me semble qu'on avait évoqué la possibilité de les supprimer, mais nous n'en avons pas reparlé au dernier point. |
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. |
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 |
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. |
…n DSFR * Mise à jour des styles active/focus du logo du header
…n DSFR * Fix: Logo hover background overflowing on mobile layout
à changer:
The text was updated successfully, but these errors were encountered: