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

Add Settings panel to change language manually, + minor fixes #62

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

hippalectryon-0
Copy link
Contributor

@hippalectryon-0 hippalectryon-0 commented Jan 17, 2023

Changement majeur: ajout d'un panel "Settings" qui contient un bouton pour changer manuellement de langue (utilisation: les gens comme moi qui ont un téléphone en anglais)

NOTE: je n'ai pas trouvé de meilleure option avec ta librairie de localisation i18n-js que de reload l'appli pour appliquer le changement (je suis habitué à i18n-next, qui propose une fonction pour ça)

Changements mineurs: fix de traductions en anglais, un peu de lint, fix de la taille d'un paragraphe dans about

N'hésite surtout pas à modifier si l'UI ne te va pas ^^

Copy link
Owner

@sallesma sallesma left a comment

Choose a reason for hiding this comment

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

Merci pour la suggestion, j'ai mis quelques commentaires sur la forme pour ne pas les perdre, mais je suis surtout un peu indécis sur le fond.

J'aimerais bien garder l'app aussi épurée que possible, à la fois côté interface et côté code, et l'ajout d'un bouton sur la Home + un écran juste pour changer de langue me parait un peu trop. J'ai l'impression que c'est une béquille pour pallier au manque de questions sur la version anglaise, plutôt qu'une vraie fonctionnalité utile. Dans un monde où toutes les questions sont traduites, j'ai l'impression que cette fonctionnalité n'est pas nécessaire

(ceci dit, je vais inclure via un cherry-pick les corrections de traductions que tu as faite en même temps)

src/screens/HomeScreen.js Outdated Show resolved Hide resolved
src/screens/SettingsScreen.js Outdated Show resolved Hide resolved
@hippalectryon-0
Copy link
Contributor Author

Remarque: sur l'écran d'accueil, Il est noté "82 questions" même en français ! J'essaie de voir où sont passées les questions manquantes

rewrite getRandomElementsFromArray
@hippalectryon-0
Copy link
Contributor Author

^ Fixed

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.

2 participants