Problème avec la définition de langue par défaut selon la configuration du navigateur

dtrucs commented 2 years ago

Actuellement la configuration permet de définir des langues prises en charges par l'application et celle par défaut.
Par exemple:

    "supportedLanguages": ["fr", "en", "it"],
    "defaultLanguage": "fr"

L'app va d'abord chercher si la locale par défaut est configurée dans le navigateur du client :

si oui le client va avoir l'interface dans cette langue
si non, l'app va chercher une correspondance entre supportedLanguages et celles configurées dans le navigateur pour lui délivrer une page dans cette langue.
si encore non, l'app ne réfléchit pas davantage et distribue celle définit en defaultLanguage.

En me basant sur l'exemple ci-dessus et selon la configuration de mon navigateur je risque d'avoir des rendus non satisfaisant :

Mon navigateur ayant les langues ['fr', 'en'] va me rendre l'interface en français
Mon navigateur ayant les langues ['fr-fr', 'en'] va me rendre l'interface en anglais

En effet l'app ne tient pas compte de la régionalisation.

Il y a bien le helper getDefaultLanguage qui veille à la vérification de la régionalisation sauf qu'il est inutile car il est appelé toujours après useRouter qu'on peut voir ici et là dans le code useRouter().locale ?? getDefaultLanguage();.

UseRouter ne fait pas la distinction avec la région, il faudrait ajouter les langues et leurs régionalisations dans supportedLanguages ce qui n'est pas souhaitable.

En solution :

Next12 (l'app est encore en version 11) permettrait de faire un middleware https://nextjs.org/docs/advanced-features/i18n-routing#prefixing-the-default-locale où nous pourrions intégrer la logique de régionalisation.
Se passer de la détection de next/router et appeler getDefaultLanguage(); en omettant de prefixer useRouter().locale ??. À tester si ça n'induit pas d'autres faux-positifs

camillemonchicourt commented 2 years ago

OK, donc il faudrait ajouter la prise en compte la régionalisation ?

dtrucs commented 2 years ago

OK, donc il faudrait ajouter la prise en compte la régionalisation ?

J'avais soumis le ticket par inadvertence je viens de le modifier (tu es allé trop vite dans la réponse :D)

camillemonchicourt commented 2 years ago

On est passé sur Next 12 dans la dernière version de vendredi (3.5.2)

dtrucs commented 2 years ago

Top, je jeterai un oeil

babastienne commented 9 months ago

@dtrucs problème toujours d'actualité ?

dtrucs commented 9 months ago

Toujours

GeotrekCE / Geotrek-rando-v3

Problème avec la définition de langue par défaut selon la configuration du navigateur #527