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

plotly.js en français #2252

Merged
merged 6 commits into from
Jan 18, 2018
Merged

plotly.js en français #2252

merged 6 commits into from
Jan 18, 2018

Conversation

etpinard
Copy link
Contributor

Tagging @florianleger (from #856 (comment)) for review.

cc @alexcjohnson

dictionary: {
'Autoscale': 'Échelle automatique',
'Box Select': 'Sélection en boîte',
'Click to enter Colorscale titre': 'Cliquer ici pour ajouter un titre à l\'échelle de couleurs',
Copy link
Contributor Author

@etpinard etpinard Jan 15, 2018

Choose a reason for hiding this comment

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

@alexcjohnson and I are thinking about shortening this those Cliquer ici pour ajouter items to simply Ajouter, which should be clear enough. Any objections?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done in e3038cc

@alexcjohnson alexcjohnson mentioned this pull request Jan 16, 2018
@etpinard etpinard added this to the v1.33.0 milestone Jan 16, 2018
@etpinard
Copy link
Contributor Author

@florianleger auriez-vous des suggestions ou commentaires au sujet des traductions proposées ?

J'ai vu que dans votre repo, vous avez choisi tracé pour trace. Est-ce que l'utilisation de trace vous dérangerait? Personnellement, tracé me fait plus penser à path.

@florianleger
Copy link

@etpinard Tout me semble bien ! L'utilisation du terme trace me semble effectivement plus approprié ici.

@TomDemulierChevret
Copy link
Contributor

TomDemulierChevret commented Jan 18, 2018

Dans le fichier locale-en.js on trouve l'élément dateTime: '%a %b %e %X %Y'.
Je ne sais pas si cette valeur est utilisée dans plotly.js (elle n'est pas traduite dans la locale de.js).

Si cette valeur est utilisée, alors le format correct en Français serait dateTime: '%a %e %b %Y %H:%M:%S' (correspond par exemple à Jeu 18 Jan 2018 11:46:31).

Pour le terme trace, je verrais plutôt courbe ou série qui me semblent bien utilisés que trace lorsqu'on parle de graphiques et de tableaux de données.

'Reset camera to default': 'Régler la caméra à sa valeur défaut', à modifier en 'Reset camera to default': 'Régler la caméra à sa valeur par défaut',

@TomDemulierChevret
Copy link
Contributor

I just made the pull request #2261 which allow to localize auto-formatted date ticks for the x-axis.
It already contains a working example for the french locale.

@etpinard
Copy link
Contributor Author

@TomDemulierChevret merci pour vos suggestions !

À propos de trace, je l'admets, ce n'est pas une solution parfaite. Par contre, je trouve courbe ou série trop limités. Par example, nommer un nuage de points une courbe me semble erroné. Même constat pour un heatmap. J'aime bien série, mais, ce terme met peut-être trop d'emphase sur les données de la trace et pas assez sur ses attributes esthétiques.

@TomDemulierChevret
Copy link
Contributor

Effectivement courbe ne convient pas à tous les cas (j'utilise pour l'instant uniquement le scatter plot, donc je n'ai pas tout en tête).
Pour série, on retrouve le terme dans excel même lorsqu'on génère un graphique à partir du tableur. C'est pour cela qu'il me parait plus standard que trace.
Après il existe peut être certains graphes pour lesquels cette appellation n'aurait pas de sens.

Autre petite modification à faire de mon point de vue :
'Box Select': 'Sélection en boîte', => 'Box Select': 'Sélection rectangulaire', (si j'ai bien compris l'usage du bouton en question)

@etpinard
Copy link
Contributor Author

Ah bien, Excel utilise série, Allons-y avec ça dans ce cas. Merci !

@etpinard etpinard merged commit 48512eb into master Jan 18, 2018
@etpinard etpinard deleted the plotly-fr branch January 18, 2018 17:23
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.

3 participants