Skip to content
This repository was archived by the owner on Jan 20, 2025. It is now read-only.

gatsby-image to french #76

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

akashpatel1310
Copy link

No description provided.

Copy link
Contributor

@mindsers mindsers left a comment

Choose a reason for hiding this comment

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

Thanks for your PR. But I feel like some parts were done using a translation tool. Which is forbidden.
Please fix this PR and we'll merge it.


> _Note: gatsby-image is **not** a drop-in replacement for `<img />`. It’s optimized for responsive fixed width/height images and images that stretch the full-width of a container. There are also other ways to [work with images](/docs/images-and-files/) in Gatsby that don't require GraphQL._
> _Remarque: gatsby-image n'est **pas** un remplacement instantané de `<img />`. Il est optimisé pour les images réactives de largeur / hauteur fixe et les images qui s'étendent sur toute la largeur d'un conteneur. Il existe également d'autres façons de [work with images](/docs/images-and-files/) dans Gatsby qui ne nécessitent pas GraphQL._
Copy link
Contributor

Choose a reason for hiding this comment

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

Some part of this line isn't translated


A common way to source images is to install and use `gatsby-source-filesystem` to connect your local files, but other source plugins can be used as well, such as `gatsby-source-contentful`, `gatsby-source-datocms` and `gatsby-source-sanity`.
Une façon courante de générer des images est d'installer et d'utiliser `gatsby-source-filesystem` pour connecter vos fichiers locaux, mais d'autres plugins sources peuvent également être utilisés, tels que` gatsby-source-contentful`, `gatsby-source-datocms `et` gatsby-source-sanity`.
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think générer can be used for source

@@ -33,27 +33,27 @@ module.exports = {
}
```

_For in-depth install instructions, check out the docs on [Using Gatsby Image](/docs/using-gatsby-image/)._
_Pour des instructions d'installation détaillées, consultez la documentation sur [Using Gatsby Image](/docs/using-gatsby-image/)._
Copy link
Contributor

Choose a reason for hiding this comment

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

You should translate links' text also


### Gatsby image starts with a query

To feed file data in to Gatsby Image, set up a [GraphQL query](/docs/graphql-reference/) and either pass it into a component as props or write it directly in the component. One technique is to leverage the [`useStaticQuery`](/docs/use-static-query/) hook.
Pour alimenter les données du fichier dans Gatsby Image, configurez une [GraphQL query](/docs/graphql-reference/) et transmettez-la dans un composant en tant qu'accessoires ou écrivez-la directement dans le composant. Une technique consiste à tirer parti du hook [`useStaticQuery`](/docs/use-static-query/).
Copy link
Contributor

Choose a reason for hiding this comment

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

We miss the meaning of the sentence at the beginning. It would be "Pour alimenter Gatsby Image en fichier de données..." instead of "Pour alimenter les données du fichiers dans Gatsby Image". We don't want to feed the file data but Gatsby Image with file data.

@@ -118,13 +118,13 @@ file(relativePath: { eq: "images/default.jpg" }) {
}
```

Read more about fixed image queries in the [gatsby-plugin-sharp](/packages/gatsby-plugin-sharp/#fixed) README.
En savoir plus sur les requêtes d'images fixes dans le README [gatsby-plugin-sharp](/packages/gatsby-plugin-sharp/#fixed).
Copy link
Contributor

Choose a reason for hiding this comment

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

This sentence miss some words

@mindsers mindsers mentioned this pull request Apr 30, 2021
52 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants