Fix form french translation #372
Conversation
Migrate cioos form
|
Visit the preview URL for this PR (updated for commit 96b15e3): https://cioos-metadata-form-dev-258dc--pr372-370-fix-french-ui-f2b3pv0m.web.app (expires Wed, 01 Jan 2025 21:20:17 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: c9b6275cb4b6311b719349f5e25e457b5691d09c |
| @@ -121,7 +121,7 @@ class Platforms extends FormClassTemplate { | |||
| <Typography variant="h5"> | |||
| <I18n> | |||
| <En>Platforms</En> | |||
| <Fr>Platforms</Fr> | |||
| <Fr>Plateforme</Fr> | |||
There was a problem hiding this comment.
ça semble être au pluriel
src/components/Pages/Submissions.jsx
Outdated
| @@ -203,7 +203,7 @@ class Submissions extends FormClassTemplate { | |||
| obligatoires, vous pouvez soumettre vos métadonnées pour | |||
| validation en cliquant sur l’icône « soumettre pour | |||
| validation ». Vos métadonnées seront publiées lorsqu’elles | |||
| auront été validées et approuvées par un professionel du{" "} | |||
| auront été validées et approuvées par un professionel de l'{" "} | |||
There was a problem hiding this comment.
ce titre est plus large et pas juste l'OGSL et devrait s'appliquer au moins aux régions:
SIOOC du Pacific, SIOOC de l'Atlantique, Amundsen Science ...
Je me demande si on utilise pas: auront été validées et approuvées par un professionel de {" " }
| @@ -90,12 +90,12 @@ const PlatformLeftList = ({ | |||
| {platforms.length ? ( | |||
| <I18n> | |||
| <En>Platforms in this record:</En> | |||
| <Fr>Platforms dans cet enregistrement:</Fr> | |||
| <Fr>Plateforme dans cet enregistrement:</Fr> | |||
There was a problem hiding this comment.
Doit être pluriel Plateformes
| @@ -44,7 +44,7 @@ import { En, Fr, I18n } from "./I18n"; | |||
|
|
|||
| import { UserContext } from "../providers/UserProvider"; | |||
|
|
|||
| const drawerWidth = 240; | |||
| const drawerWidth = 275; | |||
There was a problem hiding this comment.
C'est pour la barre de navigation ça?
src/components/NavDrawer.jsx
Outdated
| @@ -175,6 +175,7 @@ export default function MiniDrawer({ children }) { | |||
| signIn: <I18n en="Sign in" fr="Se Connecter" />, | |||
| logout: <I18n en="Logout" fr="Déconnexion" />, | |||
| sharedWithMe: <I18n en="Shared with me" fr="Partagé avec moi" />, | |||
| envconnection: <I18n en="Connected to development database" fr="Connecté à la base de données <br /> de développement" />, | |||
There was a problem hiding this comment.
Pour suivre la convention des noms de variables en haut il serait bien d'utiliser
envConnection
src/components/NavDrawer.jsx
Outdated
| @@ -466,7 +467,7 @@ export default function MiniDrawer({ children }) { | |||
| )} | |||
| </List> | |||
| <Divider /> | |||
| {usingDevDatabase && <h5>Connected to development database</h5>} | |||
| {usingDevDatabase && <h5>{translations.envconnection}</h5>} | |||
There was a problem hiding this comment.
voir commentaire plus haut
src/regions.js
Outdated
| }, | ||
| titleFrPossessive: "de l'Observatoire Global du Saint-Laurent (OGSL)", | ||
| titleFrPossessive: " de l'Observatoire global du Saint-Laurent (OGSL)", |
There was a problem hiding this comment.
pas sur que l'espace au début est voulu
|
@kutoso @ABiehlerOGSL et @rodriguezcme! Vous pouvez voir les changements sur le site suivante: |
Fix some of the French translation.