Skip to content

Feat: hiérarchiser les fournisseurs d'identité sur la page de connexion#3913

Open
IdrissaD wants to merge 4 commits intoPnX-SI:developfrom
naturalsolutions:feat/change-main-provider-on-login-page
Open

Feat: hiérarchiser les fournisseurs d'identité sur la page de connexion#3913
IdrissaD wants to merge 4 commits intoPnX-SI:developfrom
naturalsolutions:feat/change-main-provider-on-login-page

Conversation

@IdrissaD
Copy link
Copy Markdown
Contributor

@IdrissaD IdrissaD commented Feb 4, 2026

Fixes #3912

En lien avec la PR 144 de UsersHub-authentification-module

  • transformation des blocs de connexion local-provider et des autres fournisseurs en templates
  • refonte de la logique d'affichage des fournisseurs avec une séparation en primaryProviders et secondaryProviders
  • création d'un composant details pour masquer les secondaryProviders
  • traduction de tous les éléments de la page en français et anglais

Comment thread docs/admin/authentication-custom.rst Outdated
@IdrissaD IdrissaD force-pushed the feat/change-main-provider-on-login-page branch from 48e445e to 5f8b5f1 Compare February 4, 2026 14:26
@camillemonchicourt camillemonchicourt added this to the 2.17.1 milestone Feb 4, 2026
@IdrissaD IdrissaD force-pushed the feat/change-main-provider-on-login-page branch 2 times, most recently from badce85 to 1677a68 Compare February 25, 2026 16:01
Comment thread docs/admin/authentication-custom.rst Outdated
Comment thread frontend/src/app/modules/login/login/login.component.html Outdated
@christophe-ramet
Copy link
Copy Markdown
Contributor

christophe-ramet commented Feb 27, 2026

J'ai revue et testé la PR et ça me semble top à part les petites remarques (qui sont des détails)

Pas directement lié à cette PR mais ça m'a permis de me rendre compte que le fait qu'on ne centre pas le fournisseur d'identité donne un résultat un peu moche :

image

Pour moi centrer parrait plus clean
image

On pourrait peut être profiter de cette PR qui modifie pas mal d'élément de la page pour centrer le founisseur ?

<br />
<details>
<summary>{{ 'Authentication.OtherAuthProviders' | translate }}</summary>
<ng-container *ngFor="let provider of secondaryProviders">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ce serait pas mal de mettre un br ici car quand on a juste l'auth geonature en secondary, ça fait un peu encombré

Copy link
Copy Markdown
Contributor Author

@IdrissaD IdrissaD Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je suis d'accord mais je me suis débattu avec des br dans tous les sens, y compris derrière des ngIf, dans et en-dehors des boucles, etc., sans arriver à un résultat satisfaisant dans toutes les situations. Si j'ajoute un br entre details et summary, quand l'auth GeoNature est première des secondary, c'est bien, et quand elle est deuxième :
image
Si t'as une idée d'agencement précis des br qui donne un truc sympa dans toutes les configurations je suis preneur !

Je viens de mettre deux br au-dessus de details et un entre details et summary pour couvrir les cas les plus courants :

  • auth GeoNature seule primary, un ou plusieurs secondary
  • un ou plusieurs primary autres que GeoNature, et auth GeoNature première des secondary

Dans les autres cas, c'est moins esthétique.

@IdrissaD IdrissaD force-pushed the feat/change-main-provider-on-login-page branch from 1677a68 to cb837ea Compare February 27, 2026 14:52
@IdrissaD
Copy link
Copy Markdown
Contributor Author

Tout à fait d'accord avec le centrage, mais idem que pour les sauts de ligne, j'avais déjà essayé de centrer dans toutes les balises possibles et j'ai pas réussi... Mes limites en CSS sont très rapidement atteintes haha
Si tu peux me dire ce que t'as ajouté, ou l'ajouter toi-même, je veux bien !

@IdrissaD IdrissaD force-pushed the feat/change-main-provider-on-login-page branch from cb837ea to 139c129 Compare February 27, 2026 15:04
@christophe-ramet
Copy link
Copy Markdown
Contributor

christophe-ramet commented Feb 27, 2026

Top merci.

Ce que je te propose de mettre :

                <div>
                  <ng-container *ngFor="let provider of primaryProviders">
                    <div class="text-center">
                      <ng-container
                        *ngTemplateOutlet="providerTemplate; context: { provider: provider }"
                      />
                    </div>
                    <br *ngIf="primaryProviders.length > 1"/>
                  </ng-container>

                  <ng-container *ngIf="hasSecondaryProviders">
                    <br/>
                    <details>
                      <summary>{{ 'Authentication.OtherAuthProviders' | translate }}</summary>
                      <br/>
                      <ng-container *ngFor="let provider of secondaryProviders">
                        <div class="text-center">
                          <ng-container
                            *ngTemplateOutlet="providerTemplate; context: { provider: provider }"
                          />
                        </div>
                        <br *ngIf="secondaryProviders.length > 1"/>
                      </ng-container>
                    </details>
                  </ng-container>
                </div>

Et ça donne ça dans les différents cas :

image image image

@IdrissaD IdrissaD force-pushed the feat/change-main-provider-on-login-page branch 2 times, most recently from a3c3b5b to 0117d62 Compare February 27, 2026 16:00
@IdrissaD
Copy link
Copy Markdown
Contributor Author

Si simple, merci !
J'en ai profité pour ajouter une condition sur les br post-fournisseur afin de pas les afficher si c'est le dernier fournisseur de la liste.

Copy link
Copy Markdown
Contributor

@christophe-ramet christophe-ramet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est testé et bon pour moi merci ! :)

@jacquesfize jacquesfize mentioned this pull request Mar 9, 2026
8 tasks
@jacquesfize jacquesfize force-pushed the feat/change-main-provider-on-login-page branch from 0117d62 to f65c664 Compare March 18, 2026 14:14
@jacquesfize jacquesfize modified the milestones: 2.17.1, 2.17.2 Mar 30, 2026
@edelclaux edelclaux force-pushed the feat/change-main-provider-on-login-page branch from f65c664 to ad260a1 Compare March 31, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants