entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 9 Releases Wiki Activity

bouches-du-rhone: add template for ADFS connexion button (#81416) #390

Merged
bdauvergne merged 1 commits from wip/81416-bouches-du-rhone-bouton-pour-la into main 2023-09-25 16:47:05 +02:00
Conversation 1 Commits 1 Files Changed 3 +43
bdauvergne commented 2023-09-20 17:09:40 +02:00
Owner
Copy Link
No description provided.
bdauvergne added 1 commit 2023-09-20 17:09:40 +02:00
gitea/publik-base-theme/pipeline/head There was a failure building this commit Details
c9b31f4fed bouches-du-rhone: add template for ADFS connexion button (#81416)
fpeters reviewed 2023-09-20 17:37:33 +02:00
templates/variants/bouches-du-rhone/authentic2_auth_saml/login_cd13-adfs.html Outdated
@ -0,0 +5,4 @@
<form method="post">
<button name="login-saml-cd13-adfs" style="">
<img height="60" src="{{ portal_url }}assets/button:adfs" width="60" alt="M'identifier avec mon compte agent">
<span>M'identifier avec mon compte agent</span>
fpeters commented 2023-09-20 17:37:33 +02:00
Owner
Copy Link

Un lecteur d'écran lira deux fois "M'identifier avec mon compte agent", vraisemblablement l'image pourrait avoir alt="" pour éviter ça.

Un lecteur d'écran lira deux fois "M'identifier avec mon compte agent", vraisemblablement l'image pourrait avoir alt="" pour éviter ça.
bdauvergne commented 2023-09-20 19:14:23 +02:00
Author
Owner
Copy Link

Dak, j'ai corrigé.

Dak, j'ai corrigé.
bdauvergne force-pushed wip/81416-bouches-du-rhone-bouton-pour-la from c9b31f4fed to 8212b36d9c 2023-09-20 19:14:09 +02:00 Compare
bdauvergne changed title from WIP: bouches-du-rhone: add template for ADFS connexion button (#81416) to bouches-du-rhone: add template for ADFS connexion button (#81416) 2023-09-25 10:43:03 +02:00
aberriot reviewed 2023-09-25 12:02:21 +02:00
aberriot left a comment
Owner
Copy Link

Je n'ai pas réussi à tester en local (malgré l'activation d'un auth SAML dans authentic), je pense qu'il faut une config spécifique de page pour déclencher le template, est-ce que tu pourrais me partager la manip pour afficher le template custom ?

Je n'ai pas réussi à tester en local (malgré l'activation d'un auth SAML dans authentic), je pense qu'il faut une config spécifique de page pour déclencher le template, est-ce que tu pourrais me partager la manip pour afficher le template custom ?
fpeters reviewed 2023-09-25 15:03:58 +02:00
static/bouches-du-rhone/_custom.scss Outdated
@ -155,0 +155,4 @@
#login-page {
display: flex;
padding-bottom: 1ex;
fpeters commented 2023-09-25 15:03:58 +02:00
Owner
Copy Link

Noté par le CD13 ça casse le responsive. (#81566)

Quelle était l'intention de toute cette partie CSS ? (plutôt que laisser le fonctionnement de base ?).

Si tout ça est nécessaire, alors sur petit écran ajouter un flex-direction: vertical.

Noté par le CD13 ça casse le responsive. (#81566) Quelle était l'intention de toute cette partie CSS ? (plutôt que laisser le fonctionnement de base ?). Si tout ça est nécessaire, alors sur petit écran ajouter un flex-direction: vertical.
bdauvergne commented 2023-09-25 15:41:32 +02:00
Author
Owner
Copy Link

C'était pour centre la partie mot de passe et donner la même hauteur à la partie SAML, mais effectivement je n'y arrive en étant responsive et ça n'était pas une demande, je retire ça.

C'était pour centre la partie mot de passe et donner la même hauteur à la partie SAML, mais effectivement je n'y arrive en étant responsive et ça n'était pas une demande, je retire ça.
bdauvergne commented 2023-09-25 15:31:22 +02:00
Author
Owner
Copy Link

Je n'ai pas réussi à tester en local (malgré l'activation d'un auth SAML dans authentic), je pense qu'il faut une config spécifique de page pour déclencher le template, est-ce que tu pourrais me partager la manip pour afficher le template custom ?

Je pense qu'il suffit d'une configuration auth SAML qui a pour slug "cd13-adfs".

PS: comme là https://connexion-departement13.test.entrouvert.org/manage/authenticators/3/detail/

> Je n'ai pas réussi à tester en local (malgré l'activation d'un auth SAML dans authentic), je pense qu'il faut une config spécifique de page pour déclencher le template, est-ce que tu pourrais me partager la manip pour afficher le template custom ? Je pense qu'il suffit d'une configuration auth SAML qui a pour slug "cd13-adfs". PS: comme là https://connexion-departement13.test.entrouvert.org/manage/authenticators/3/detail/
👍 1
bdauvergne force-pushed wip/81416-bouches-du-rhone-bouton-pour-la from 8212b36d9c to 38f5b75989 2023-09-25 15:42:10 +02:00 Compare
aberriot approved these changes 2023-09-25 15:51:59 +02:00
aberriot left a comment
Owner
Copy Link

Effectivement, avec le bon slug pour l'authenticateur saml j'arrive à charger le template et avec les dernières modifs ça rend correctement sur desktop et écran étroit chez moi :)

Effectivement, avec le bon slug pour l'authenticateur saml j'arrive à charger le template et avec les dernières modifs ça rend correctement sur desktop et écran étroit chez moi :)
bdauvergne merged commit 38f5b75989 into main 2023-09-25 16:47:05 +02:00
bdauvergne deleted branch wip/81416-bouches-du-rhone-bouton-pour-la 2023-09-25 16:47:05 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
aberriot
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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#390
No description provided.
Delete Branch "wip/81416-bouches-du-rhone-bouton-pour-la"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?