etalab / transport-site

Rendre disponible, valoriser et améliorer les données transports
https://transport.data.gouv.fr
184 stars 28 forks source link

Modifications accueil et menu #4016

Closed AntoineAugusti closed 1 day ago

AntoineAugusti commented 1 week ago

Fixes #3914

Implémente les changements détaillés dans https://github.com/etalab/transport-site/issues/3914#issuecomment-2181097607

Le bouton pour l'espace réutilisateur reste masqué tant qu'il n'est pas sorti pour le grand public. Tenter d'accéder aux espaces utilisateurs sans être connecté amène sur une landing page d'information, présentant la fonctionnalité et demandant de se connecter.

image Screenshot 2024-06-21 at 11 24 37

AurelienC commented 1 week ago

Lié également à https://github.com/etalab/transport-site/issues/3566

cyrilmorin commented 1 week ago

OK avec @vdegove sur le nombre d'onglets et le menu à rallonge.

Plutôt emballé par la suppression du nom d'utilisateur en mode connecté (en le mettant en info bulle sur le logo "user") ET un logo "silhouette impersonnelle" en mode déconnecté

L'onglet statistique devra exister un jour ou l'autre (le plus tôt possible) avec nos futurs super TDB et dataviz sur lesquels on doit bosser !

AntoineAugusti commented 1 week ago

Je n'ai pas compris ce qu'on devait changer.

Supprimer le menu "Statistiques" pour le moment ? Quel est le changement à effectuer pour le menu utilisateur en mode connecté ? (actuellement nom + prénom + avatar)

cyrilmorin commented 1 week ago

Sorry, ça devait pas être clair (TGIF ...) suite à échange sur Mattermost On conserve l'onglet statistiques et le fonctionnement actuel pour la connexion (on étudiera les options pour diminuer le nombre d'onglets ultérieurement)

vdegove commented 1 week ago

Attention : je n’étais peut-être pas clair, mais ce que je suggérais pour le mode «connecté», c’est de rien changer au fonctionnement, juste mettre un petit logo à gauche du nom d’utilisateur, pour que visuellement ça soit séparé des autres menus.

Donc, de la même manière qu’il y a un petit logo entre « blog » et « se connecter », en mettre un entre « blog » et « Vincent Degove » :

image image

cyrilmorin commented 1 week ago

OK je comprends mieux, c'est effectivement moins impactant :) OK pour moi !

AntoineAugusti commented 5 days ago

@vdegove Le menu actuel affiche déjà l'avatar à côté de son nom. Ce n'est pas affiché pour toi car tu n'as pas d'avatar.

image

https://github.com/etalab/transport-site/blob/25c9356043ac1cf49359c1635edc804d0df549fe/apps/transport/lib/transport_web/templates/layout/_header.html.heex#L80-L82

juste mettre un petit logo à gauche du nom d’utilisateur, pour que visuellement ça soit séparé des autres menus.

Peut-on considérer que c'est déjà fait ? Tu voudrais un avatar par défaut ?

vdegove commented 4 days ago

Peut-on considérer que c'est déjà fait ? Tu voudrais un avatar par défaut ?

On pourrait

  1. déplacer l’avatar à gauche du nom (ce qui d’ailleurs serait cohérent avec le menu «se connecter» où effectivement c’est à gauche)
  2. oui, avoir un avatar par défaut, ou récupérer le faux avatar de datagouv

image

Je pinaille un peu, mais je trouve que cette séparation visuelle entre les vrais liens de navigation et les liens type « menu utilisateur / langue » est importante dans le cadre d’un menu de navigation principal à rallonge.

AntoineAugusti commented 4 days ago

@vdegove Aucun problème, je pense que c'est une bonne idée. Je note de fallback sur l'avatar par défaut si l'utilisateur n'en a pas, et de déplacer à gauche du nom.

Ce sera aussi cohérent avec l'affichage sur data.gouv.fr et les discussions sur notre plateforme où on affiche l'avatar par défaut de data.gouv.fr si l'utilisateur n'en a pas un.

AntoineAugusti commented 4 days ago

J'ai modifié la position (à gauche du nom de l'utilisateur) dans le menu et mis l'avatar généré automatiquement quand l'utilisateur n'a pas un avatar personnalisé.

image

image