adapt a2 accounts page to service appearance customization (#77688) #271

Merged
pmarillonnet merged 1 commits from wip/77688-a2-accounts-service-customization into main 2023-06-13 15:16:52 +02:00
Owner
No description provided.
pmarillonnet force-pushed wip/77688-a2-accounts-service-customization from 971af00d55 to b2a1237e1f 2023-05-17 10:26:10 +02:00 Compare
Author
Owner
No description provided.
pmarillonnet changed title from WIP: adapt a2 accounts page to service appearance customization (#77688) to adapt a2 accounts page to service appearance customization (#77688) 2023-05-17 10:27:11 +02:00
Author
Owner

Vu en privé avec Mikaël et ThomasJ. On ne va pas faire un bloc supplémentaire, simplement ajouter un haut de contenu comme cela peut déjà être le cas avec les pages de login et d’autorisation. Nouvelle proposition.

Vu en privé avec Mikaël et ThomasJ. On ne va pas faire un bloc supplémentaire, simplement ajouter un haut de contenu comme cela peut déjà être le cas avec les pages de login et d’autorisation. Nouvelle proposition.
pmarillonnet force-pushed wip/77688-a2-accounts-service-customization from b2a1237e1f to 4281c02c71 2023-05-17 14:59:02 +02:00 Compare
smihai reviewed 2023-05-17 15:07:30 +02:00
@ -8,0 +25,4 @@
{% firstof generic_service_name service.name service.ou.name as name %}
{% firstof generic_service_logo_url service_logo_url service_ou_logo_url as logo_url %}
<div id="a2-service-information" class="service-message">
<h2>Connexion depuis le service {{ name }}</h2>
Owner

Vu qu'on est déjà connecté quand on arrive sur cette page ne faudrait-il pas plutôt indiquer "Authentification depuis le service ..." ou "Vous etes authentifies depuis le service ...".

Vu qu'on est déjà connecté quand on arrive sur cette page ne faudrait-il pas plutôt indiquer "Authentification depuis le service ..." ou "Vous etes authentifies depuis le service ...".
Author
Owner

D’ac oui, j’ai corrigé dans la branche avec la première des deux propositions que tu fais ici.

D’ac oui, j’ai corrigé dans la branche avec la première des deux propositions que tu fais ici.
Owner

Honnêtement je comprends très peu le sens de "Authentification depuis le service ..."; sur les pages de connexion on écrit "Connexion depuis ..." pour rassurer l'usager sur le fait qu'il est bien dans le parcours qu'il a initié en cliquant "se connecter" sur un site graphiquement tout différent. Là, on peut lire le "Connexion depuis XXX." comme "Connexion initiée sur le service XXX."

Mais ici, quelle fonction "Authentification depuis le service ..." et quelle lecture possible ? L'utilité ici, il me semble, c'est surtout permettre un retour sur le site "XXX".

Je ne vois pas la fonction où "Authentification initiée depuis le service XXX" aurait un sens. :/

Honnêtement je comprends très peu le sens de "Authentification depuis le service ..."; sur les pages de connexion on écrit "Connexion depuis ..." pour rassurer l'usager sur le fait qu'il est bien dans le parcours qu'il a initié en cliquant "se connecter" sur un site graphiquement tout différent. Là, on peut lire le "Connexion depuis XXX." comme "Connexion initiée sur le service XXX." Mais ici, quelle fonction "Authentification depuis le service ..." et quelle lecture possible ? L'utilité ici, il me semble, c'est surtout permettre un retour sur le site "XXX". Je ne vois pas la fonction où "Authentification initiée depuis le service XXX" aurait un sens. :/
Author
Owner

Honnêtement je comprends très peu le sens de "Authentification depuis le service ..."; sur les pages de connexion on écrit "Connexion depuis ..." pour rassurer l'usager sur le fait qu'il est bien dans le parcours qu'il a initié en cliquant "se connecter" sur un site graphiquement tout différent. Là, on peut lire le "Connexion depuis XXX." comme "Connexion initiée sur le service XXX."

Mais ici, quelle fonction "Authentification depuis le service ..." et quelle lecture possible ? L'utilité ici, il me semble, c'est surtout permettre un retour sur le site "XXX".

Je ne vois pas la fonction où "Authentification initiée depuis le service XXX" aurait un sens. :/

Ok, je n’avais pas saisi cette subtilité dans l’intitulé choisi. J’ai poussé une nouvelle version avec simplement “Connexion depuis …".

> Honnêtement je comprends très peu le sens de "Authentification depuis le service ..."; sur les pages de connexion on écrit "Connexion depuis ..." pour rassurer l'usager sur le fait qu'il est bien dans le parcours qu'il a initié en cliquant "se connecter" sur un site graphiquement tout différent. Là, on peut lire le "Connexion depuis XXX." comme "Connexion initiée sur le service XXX." > > Mais ici, quelle fonction "Authentification depuis le service ..." et quelle lecture possible ? L'utilité ici, il me semble, c'est surtout permettre un retour sur le site "XXX". > > Je ne vois pas la fonction où "Authentification initiée depuis le service XXX" aurait un sens. :/ Ok, je n’avais pas saisi cette subtilité dans l’intitulé choisi. J’ai poussé une nouvelle version avec simplement “Connexion depuis …".
Author
Owner

Ok, je n’avais pas saisi cette subtilité dans l’intitulé choisi. J’ai poussé une nouvelle version avec simplement “Connexion depuis …".

Et en fait j’ai lu trop rapidement, c’est le fait même de faire apparaître une telle phrase qui pose problème. On peut retirer l’intitulé oui, ça ne semble par primordial ici.

> Ok, je n’avais pas saisi cette subtilité dans l’intitulé choisi. J’ai poussé une nouvelle version avec simplement “Connexion depuis …". Et en fait j’ai lu trop rapidement, c’est le fait même de faire apparaître une telle phrase qui pose problème. On peut retirer l’intitulé oui, ça ne semble par primordial ici.
Author
Owner

Et en fait j’ai lu trop rapidement, c’est le fait même de faire apparaître une telle phrase qui pose problème. On peut retirer l’intitulé oui, ça ne semble par primordial ici.

Vu avec Mikaël en direct, je vais pousser la version sans intitulé.
L’important est que l’usager dispose d’un lien pour revenir au service depuis lequel il s’est connecté.

> Et en fait j’ai lu trop rapidement, c’est le fait même de faire apparaître une telle phrase qui pose problème. On peut retirer l’intitulé oui, ça ne semble par primordial ici. Vu avec Mikaël en direct, je vais pousser la version sans intitulé. L’important est que l’usager dispose d’un lien pour revenir au service depuis lequel il s’est connecté.
pmarillonnet force-pushed wip/77688-a2-accounts-service-customization from 4281c02c71 to 20b9078319 2023-05-17 15:27:22 +02:00 Compare
smihai approved these changes 2023-05-17 15:45:18 +02:00
smihai left a comment
Owner

Go

Go
fpeters reviewed 2023-05-21 10:37:50 +02:00
@ -8,0 +32,4 @@
{% if service.home_url or generic_service_home_url %}
<a href="{% firstof service.home_url generic_service_home_url %}">
{% endif %}
<img src="{{ logo_url }}" alt="{{ name }}"/>
Owner

Ça a été validé et ça me va si ça passe ainsi, mais s'il y a le temps, si ça vise le prochain cycle, il faudrait plutôt un alt="" ici, l'image n'apporte rien de bon, la lecture actuelle serait "Authentification depuis le service XXX. XXX.", c'est mieux de s'épargner la répétition.

Ça a été validé et ça me va si ça passe ainsi, mais s'il y a le temps, si ça vise le prochain cycle, il faudrait plutôt un alt="" ici, l'image n'apporte rien de bon, la lecture actuelle serait "Authentification depuis le service XXX. XXX.", c'est mieux de s'épargner la répétition.
Author
Owner

Ok, je fais la modification.

Ok, je fais la modification.
pmarillonnet force-pushed wip/77688-a2-accounts-service-customization from 20b9078319 to 6d3d97eefa 2023-05-30 14:33:17 +02:00 Compare
pmarillonnet force-pushed wip/77688-a2-accounts-service-customization from 6d3d97eefa to d4869c01f9 2023-06-13 12:00:17 +02:00 Compare
pmarillonnet force-pushed wip/77688-a2-accounts-service-customization from d4869c01f9 to a7d5ca22ad 2023-06-13 14:05:12 +02:00 Compare
pmarillonnet changed title from adapt a2 accounts page to service appearance customization (#77688) to WIP: adapt a2 accounts page to service appearance customization (#77688) 2023-06-13 14:19:46 +02:00
Author
Owner

Vu avec Mikaël en privé, la bonne manière d’intégrer cela est d’altérer, pour le thème CD24 en particulier le header, qui prendrait la couleur, le lien et le logo choisis pour la configuration “Compte unique de territoire”. Je vais voir si on peut proposer une modification générique ou bien s’il faut toucher au thème CD24 en particulier.

Vu avec Mikaël en privé, la bonne manière d’intégrer cela est d’altérer, pour le thème CD24 en particulier le header, qui prendrait la couleur, le lien et le logo choisis pour la configuration “Compte unique de territoire”. Je vais voir si on peut proposer une modification générique ou bien s’il faut toucher au thème CD24 en particulier.
pmarillonnet force-pushed wip/77688-a2-accounts-service-customization from a7d5ca22ad to 9ee969e9cd 2023-06-13 15:06:48 +02:00 Compare
pmarillonnet changed title from WIP: adapt a2 accounts page to service appearance customization (#77688) to adapt a2 accounts page to service appearance customization (#77688) 2023-06-13 15:15:29 +02:00
Author
Owner

Vu avec Mikaël en privé, la bonne manière d’intégrer cela est d’altérer, pour le thème CD24 en particulier le header, qui prendrait la couleur, le lien et le logo choisis pour la configuration “Compte unique de territoire”. Je vais voir si on peut proposer une modification générique ou bien s’il faut toucher au thème CD24 en particulier.

En fait non, incompréhension de ma part, on reste sur ce qui est fait de façon générique ici. On reste, modulo une petite variation du titre, sur la version validée, que je vais donc pousser.

> Vu avec Mikaël en privé, la bonne manière d’intégrer cela est d’altérer, pour le thème CD24 en particulier le header, qui prendrait la couleur, le lien et le logo choisis pour la configuration “Compte unique de territoire”. Je vais voir si on peut proposer une modification générique ou bien s’il faut toucher au thème CD24 en particulier. En fait non, incompréhension de ma part, on reste sur ce qui est fait de façon générique ici. On reste, modulo une petite variation du titre, sur la version validée, que je vais donc pousser.
pmarillonnet merged commit b7dc96fb7f into main 2023-06-13 15:16:52 +02:00
pmarillonnet deleted branch wip/77688-a2-accounts-service-customization 2023-06-13 15:16:53 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#271
No description provided.