Translation for "Dom Elements" #37
Conversation
Nab69
added
the
Pending Review
|
Deploy preview for fr-reactjs ready! Built with commit 95c3b77 |
|
Deploy preview for fr-reactjs ready! Built with commit c5a65eb |
|
Merci @sheplu pour cette première relecture. Je me suis aussi trouvé un faux-ami en rebasant une phrase en conflit avec master ! Normalement tout est corrigé. |
tdd
added
Pending Review
There was a problem hiding this comment.
Salut @damusnet !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de nous filer un coup de main. Et purée, c'est vraiment du super boulot, je suis hyper impressionné !
Bon, évidemment, comme je suis moi, j'ai quand même trouvé des trucs à redire, des petits détails pour l'essentiel, mais un ou deux vrais sujets aussi.
Une fois ma review traitée, je ferai comme toujours une dernière repasse visuelle, dans laquelle je veillerai notamment aux insécables, que tu as semble-t-il totalement omis 😉
Tout est fait sous forme de Suggestions GitHub, il te suffit à chaque suggestion que tu approuves de cliquer le bouton d'ajout au lot de modifs à committer, et sur la fin de créer le commit groupé. (ce mode de regroupement en un seul commit est à favoriser absolument, mais tu devras aller dans l'onglet Files changed pour y accéder, il n'est pas utilisable dans l’onglet Conversation)
Si certaines appellent discussion, n'hésite pas, tu réponds, on en parle, on avance ensemble 🤝
Quand tout sera bouclé côté code, je ferai une repasse visuelle finale en local avant de fusionner.
Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans master !
tdd
added
Reviewed
and removed
Pending Review
|
@damusnet ce serait vraiment cool que tu aies le temps de traiter mes retours aujourd'hui ; on essaie de boucler le Core pour ce soir… |
|
@tdd Je regarde ça maintenant ! Merci pour les retours. |
Co-Authored-By: damusnet <damien@varron.fr>
There was a problem hiding this comment.
@tdd Merci beaucoup pour toutes tes corrections et les commentaires attachés. J'apprécie énormément le temps et le soin que tu prends à cet effet.
J'ai intégré toutes tes suggestions et mis à jour le title de la nav et de ma page. Préviens moi si j'ai omis quoi que ce soit.
Vraiment désolé pour les espaces insécables ; j'étais persuadé d'avoir copié / collé depuis la doc, mais j'ai du rater quelque chose.
| </div> | ||
| ``` | ||
|
|
||
| Not all style properties are converted to pixel strings though. Certain ones remain unitless (eg `zoom`, `order`, `flex`). A complete list of unitless properties can be seen [here](https://github.com/facebook/react/blob/4131af3e4bf52f3a003537ec95a1655147c81270/src/renderers/dom/shared/CSSProperty.js#L15-L59). | ||
| Toutes les propriétés de style ne sont pas systématiquement converties en pixels pour autant. Certaines restent sans unité (e.g. `zoom`, `order`, `flex`). La liste complète des propriétés sans unité peut-être consultée [ici](https://github.com/facebook/react/blob/4131af3e4bf52f3a003537ec95a1655147c81270/src/renderers/dom/shared/CSSProperty.js#L15-L59). | ||
|
|
||
| ### suppressContentEditableWarning {#suppresscontenteditablewarning} |
There was a problem hiding this comment.
Peut-être que t'as jamais essayé d'injecter une iframe dans un noscript dans une app SSR ? 😂
Bonjour,
Voici ma proposition de traduction de la page DOM Elements.
\o/