Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Ajuster le wording et le design des pages d'authentification (PIX-15198) #10502

Conversation

EmmanuelleBonnemay
Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay commented Nov 6, 2024

🍂 Problème

Il faut traiter les retours de la recette des nouvelles pages d'authentification

🌰 Proposition

Traiter les points soulevés dans https://1024pix.atlassian.net/wiki/spaces/DA/pages/4647419930/Recette+des+mires+de+connexion#Phase-de-recette-2

  1. Page d’inscription: pour l'input d’adresse email, en Français uniquement, avoir le placeholder suivant: [email protected] au lieu de [email protected]
  2. Pages de connexion, inscription, réinitialisation de mot de passe : Pour l’input d’adresse e-mail (page de connexion & inscription et réinitialisation du mot de passe), en Anglais, avoir le placeholder suivant :
    e.g [email protected]
  3. Page des SSO: Bug sur mobile, le menu du dropdown est trop large (+ large que la fenetre)
  4. Page des SSO: Sur le choix de l'organisation (dans un contexte d'inscription), sur le bouton il est écrit "Je me connecte" au lieu de "Je m'inscris" 
  5. Page des SSO: Dans le placeholder on parle d’”organisme” au lieu de d’organisation, replacer par “Sélectionner une organisation”
  6. Page de réinitialisation de mot de passe: [Français & Anglais] on a le bouton "Se connecter sur Pix" au lieu de "Se connecter"
  7. Inconsistence entre les pages “Mot de passe oublié” et “Choix des SSO” sur l’affichage du sous-texte: Faire comme dans l'écran de “Choix des SSO” (à gauche et même taille et couleurs de texte) ; vérifier aussi que la marge supérieure est identique au sous-teste de la page de choix des SSO (32px au lieu de 16px)

🎃 Remarques

Les clés de traduction mériteront peut-être d'être triées / réorganisées dans un autre ticket.

🪵 Pour tester

Vérifier les points évoqués dans https://1024pix.atlassian.net/wiki/spaces/DA/pages/4647419930/Recette+des+mires+de+connexion#Phase-de-recette-2

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-15198-improve-authentication-pages-wording-and-design branch from de4c866 to 30a3b33 Compare November 7, 2024 07:36
}


}
Copy link
Contributor

Choose a reason for hiding this comment

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

Mettre la même marge entre le bouton et le texte que pour la page des SSO:

image

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Modification faite !

Copy link
Contributor

@er-lim er-lim left a comment

Choose a reason for hiding this comment

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

Lu et testé en RA sur firefox ✅

@@ -115,15 +114,11 @@ export default class PasswordResetDemandForm extends Component {
</div>

<p class="authentication-password-reset-demand-form__help">
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggestion utiliser plutôt une balise section qui me paraît plus approprié pour ce bloc 😄

Suggested change
<p class="authentication-password-reset-demand-form__help">
<section class="authentication-password-reset-demand-form__help">

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Modification faite, merci !
J'ai aussi ajouté un white-space: normal qui règle le cas des noms très longs. (j'ai trouvé le nowrap mais pas le wrap !)

@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-15198-improve-authentication-pages-wording-and-design branch 2 times, most recently from 8ca2baa to f0629fa Compare November 7, 2024 13:56
Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

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

Parfait ✅

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15198-improve-authentication-pages-wording-and-design branch from f0629fa to 6c89f16 Compare November 7, 2024 16:11
@pix-service-auto-merge pix-service-auto-merge merged commit 80aeeac into dev Nov 7, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15198-improve-authentication-pages-wording-and-design branch November 7, 2024 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants