Produits Entr'ouvert » Authentic 2

Thomas Noël a écrit :

Dans « <p title="{{ identifier.issuer }}">{% trans "Link with SAML identity provider created on" %} {{ identifier.created }}</p> » j'ai du mal à voir à quoi ça va ressemble, le title= il va être visible ou c'est de la popup ?

C'est visible au survol, dans l'idée que c'est une URL moche et qu'on a pas forcément envie d'alourdir la lecture de la page avec (alors que j'affiche le nom du fournisseur OIDC qui je suppose est plus lisible). Mais si tu me dis que ces considérations esthétiques n'ont pas lieu d'être, j'affiche volontiers la totale (alternativement, tout afficher seulement si request.user.is_superuser ?).

Valentin Deniaud a écrit :

C'est visible au survol, dans l'idée que c'est une URL moche et qu'on a pas forcément envie d'alourdir la lecture de la page avec (alors que j'affiche le nom du fournisseur OIDC qui je suppose est plus lisible). Mais si tu me dis que ces considérations esthétiques n'ont pas lieu d'être, j'affiche volontiers la totale (alternativement, tout afficher seulement si request.user.is_superuser ?).

J'aurais vu des sections pliables (pliées par defaut) avec un intitulé du genre "Link with SAML/OIDC provider", avec les détails dont parle Thomas quand on déplie la section.

Il faudrait sûrement un peu de réflexion sur l'érgonomie de la page d'un compte.

(j'avoue avoir assez peu de considérations esthétiques sur cette page ;) )

C'est mieux de mettre des screenshots si vous voulez un avis esthétique.

Ma seule remarque serait de remplacer "Creation date:" par "Created on:" parce que 1) c'est plutôt une date+heure et 2) ça sera plus court en français.

On n'a pas de libellé à afficher à la place de l'URL "issuer" ?

Ça pourrait être juste dans la continuité du reste, pas dans une section dédiée, comme l'est l'info FranceConnect ?

ex: Liaison à "nom de l'idp" créée le 25 mars 2021 14:47. (+ une parenthèse avec le NameID si c'est pertinent).

Frédéric Péters a écrit :

On n'a pas de libellé à afficher à la place de l'URL "issuer" ?

Si, dans le code il y a {% firstof identifier.idp.DISPLAY_NAME identifier.issuer %} qui s'en occupe (mais dans mon screenshot pas des display_name donc l'URL est affichée). Code repris de ce qu'on affiche déjà en frontoffice.

Ça pourrait être juste dans la continuité du reste, pas dans une section dédiée, comme l'est l'info FranceConnect ?

J'avais fait comme ça avant d'appliquer la remarque de Serghei (#28210#note-7), il faudrait trancher.

il faudrait trancher

OK désolé juste réagi sur la capture; je veux bien trancher dans ma direction; voilà.

Alternativement, ce serait d'également passer les infos FranceConnect sous ce sous-titre; mais pas le mix actuel.

Je tranche pour l'option alternative de Frédéric: passer les infos de liens avec des IDPs externes sous le sous-titre. Et dans ce cas le renommer en "Liens externes".
Un jour on y ajoutera les infos d'autres IDPs OIDC.

Déplié par défaut alors, sur l'idée donc qu'il n'y ait pas beaucoup de lignes, et qu'aujourd'hui l'info FranceConnect était directement visible. (non ?)

Frédéric Péters a écrit :

Déplié par défaut alors, sur l'idée donc qu'il n'y ait pas beaucoup de lignes, et qu'aujourd'hui l'info FranceConnect était directement visible. (non ?)

J'aurai dit de faire simple et de tout mettre au même niveau, comme le dit Valentin jamais personne n'a plusieurs liaison, donc juste afficher "Lié à FranceConnect/à IdP de Montargis/en SAML à https://trucmuche (ici le NameID/le sub OIDC sauf pour FC)" ça me parait suffisant, sans fioritures, je ne suis pas sûr que la date de création ce soit utile, ou au pire en <span title="créé le ....">..</span>.

Valentin a fait ça, Serghei lui a demandé de changer, j'ai réagi sans lire l'historique pour suggérer de faire comme la ligne FC, Valentin a demandé que ça soit tranché, je suis resté sur mon idée mais j'ai noté que ce qui comptait d'abord c'était la cohérence, là-dessus Serghei a suggéré l'option actuelle, et Valentin il fait et refait son patch.

Frédéric Péters a écrit :

Valentin a fait ça, Serghei lui a demandé de changer, j'ai réagi sans lire l'historique pour suggérer de faire comme la ligne FC, Valentin a demandé que ça soit tranché, je suis resté sur mon idée mais j'ai noté que ce qui comptait d'abord c'était la cohérence, là-dessus Serghei a suggéré l'option actuelle, et Valentin il fait et refait son patch.

Ben c'est Serghei qui a tort pas moi.

Ok, j'ai tort. Avançons.
Je suis ok pour le patch proposé dans #28210#note-11.

Serghei Mihai a écrit :

Ok, j'ai tort. Avançons.
Je suis ok pour le patch proposé dans #28210#note-11.

Le patch que tu pointes contient les sections dépliables (enfin c'est écrit dans sa description) avec leur titre un peu lourd, je dis justement que c'est une présentation un peu too much donc plutôt le patch #28210#note-4 avec les ajouts demandés par Thomas en title=".." ou alors un truc foldable mais avec les informations directement dans le noeud legend parce "Link with OIDC provider" ça prend trop d'espace pour trop peu d'information, donc soit ça :

<p title="{% blocktrans date=user.oidc_account.created %}created on {{ date }}{% enblocktrans %}">Link with {{ user.oidc_account.provider.name }} (sub {{ user.oidc_account.sub }})</p>

<legend>Link with {{ user.oidc_account.provider.name }} (sub {{ user.oidc_account.sub }})</legend>
....

Avec toutes mes excuses si je suis un peu trop chiant.