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

UX improvements for XSSS #453

Merged
merged 9 commits into from
Mar 16, 2023
Merged

UX improvements for XSSS #453

merged 9 commits into from
Mar 16, 2023

Conversation

aulamber
Copy link
Contributor

@aulamber aulamber commented Mar 9, 2023

Cf. #442
and #451

Todos:

  • Change buttons "Configurer" and "Activer la sauvegarde automatique des messages"
    into "Activer"
  • Le texte au dessous de "Sauvegarde automatique des messages" devrait etre un générique:
    "Cette fonction vous permet de sauvegarder automatiquement vos messages et de les récupérer à tout moment à l’aide d’un Code de récupération."

-> before:
Screenshot 2023-03-14 at 15 05 14

-> after:
Screenshot 2023-03-14 at 18 05 28

  • Change button "vérifiez" en "vérifier"

-> before
Screenshot 2023-03-14 at 16 17 28

-> after
Screenshot 2023-03-14 at 17 59 52

  • Remove label "ou" and button "Envoyer"

-> before:
Screenshot 2023-03-14 at 18 07 56

-> after:
Screenshot 2023-03-14 at 18 00 08

  • both buttons "Configurer" and "Activer la sauvegarde automatique des messages" leads to the same dialog (sauvegarder votre clé de récupération). Yet they are not named the same way. Change to the name label "Activer".

-> before:
Screenshot 2023-03-14 at 18 10 49

-> after:
Screenshot 2023-03-14 at 18 05 28

  • Instead of button "button "Connecter cette session à la sauvegarde des clés", have "Vérifier cette session"

-> before:
Screenshot 2023-03-14 at 18 13 51

-> after:
Screenshot 2023-03-14 at 18 01 22

@aulamber aulamber requested a review from jdauphant March 10, 2023 10:39
Copy link
Contributor

@jdauphant jdauphant left a comment

Choose a reason for hiding this comment

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

@aulamber J'ai relu et validé les textes, j'attends les captures d'écrans.

@Caroline-lawson @areox-net si vous voulez relire les textes

@aulamber aulamber marked this pull request as draft March 14, 2023 15:07
@aulamber aulamber marked this pull request as ready for review March 14, 2023 17:14
@aulamber aulamber requested a review from odelcroi March 14, 2023 17:14
@aulamber aulamber merged commit 7bf418c into develop_tchap Mar 16, 2023
@odelcroi odelcroi linked an issue Mar 21, 2023 that may be closed by this pull request
@odelcroi odelcroi deleted the ux-improvements-for-xsss branch March 31, 2023 15:55
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.

Cross signing UX : No way of exiting when shown the recovery key
5 participants