Skip to content

2020-10-20-react-v17.md #410

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

Closed
wants to merge 8 commits into from
Closed

2020-10-20-react-v17.md #410

wants to merge 8 commits into from

Conversation

FranciscoImanolSuarez
Copy link

As I mentioned today, I leave you my PR of the document 2020-10-20-react-v17.md

I hope it's all okay!

My number # 201

👨‍🚀

@netlify
Copy link

netlify bot commented Nov 18, 2020

Deploy preview for es-reactjs ready!

Built with commit 7d76684

https://deploy-preview-410--es-reactjs.netlify.app

@carburo carburo mentioned this pull request Nov 18, 2020
Copy link
Member

@carburo carburo left a comment

Choose a reason for hiding this comment

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

¡Hola @FranciscoImanolSuarez!
Muchas gracias por la contribución.
En la mayoría de los casos has utilizado «usted» para la segunda persona del singular. Para mantener una consistencia en la documentación, hemos optado por utilizar «tú» en estos casos. Además dejé algunas sugerencias en algunas líneas. No dudes en preguntar cualquier duda o si necesitas más contexto sobre alguna de ellas.


To enable gradual updates, we've needed to make some changes to the React event system. React 17 is a major release because these changes are potentially breaking. You can check out our [versioning FAQ](/docs/faq-versioning.html#breaking-changes) to learn more about our commitment to stability.
Para habilitar las actualizaciones graduales, necesitamos realizar algunos cambios en el sistema de eventos React. React 17 es una versión importante porque estos cambios potencialmente se están rompiendo. Puede consultar nuestras [preguntas frecuentes sobre versiones](/docs/faq-versioning.html#breaking-changes) para conocer más sobre nuestro compromiso con la estabilidad.
Copy link
Member

Choose a reason for hiding this comment

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

React 17 es una versión importante porque estos cambios potencialmente se están rompiendo.

La traducción de «major» de esta forma no parece estar muy extendida e incluso propensa a confusión. Creo que preferiblemente deberíamos usar el calco «mayor» o como hace la traducción de semver.org, el anglicismo crudo.

Por otra parte la traducción «potentially breaking changes» creo que no expresa completamente el sentido de la frase original. ¿Quizá «cambios que potencialmente introduzcan rupturas» o «cambios potencialmente incompatibles» siguiendo la terminología de semver.org?


## Other Breaking Changes
## Otros cambios importantes
Copy link
Member

Choose a reason for hiding this comment

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

Otros cambios importantes
Deberíamos seguir una terminología consistente. En el caso de breaking change probablemente usaríamos «cambio incompatible» siguiendo la terminología de semver.org.

* Warn if calling setState outside of render but before commit. ([@sebmarkbage](https://github.com/sebmarkbage) in [#18838](https://github.com/facebook/react/pull/18838))
* Mejorar el mensaje de error `findByType`. ([@henryqdineen](https://github.com/henryqdineen) en [#17439](https://github.com/facebook/react/pull/17439))

### Modo concurrente (Concurrent Mode) (Experimental) {#concurrent-mode-experimental}
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
### Modo concurrente (Concurrent Mode) (Experimental) {#concurrent-mode-experimental}
### Modo concurrente (Experimental) {#concurrent-mode-experimental}

* Use delegation for `onSubmit` and `onReset` events. ([@gaearon](https://github.com/gaearon) in [#19333](https://github.com/facebook/react/pull/19333))
* Improve memory usage. ([@trueadm](https://github.com/trueadm) in [#18970](https://github.com/facebook/react/pull/18970))
* Delegar eventos a las raíces en lugar de `document`. ([@trueadm](https://github.com/trueadm) en [#18195](https://github.com/facebook/react/pull/18195) y [otros](https://github.com/facebook/react/pulls?q=is%3Apr+author%3Atrueadm+modern+event+is%3Amerged))
* Limpia todos los efectos antes de ejecutar los siguientes efectos. ([@bvaughn](https://github.com/bvaughn) en [#17947](https://github.com/facebook/react/pull/17947))
Copy link
Member

Choose a reason for hiding this comment

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

Limpia todos los efectos antes de ejecutar los siguientes efectos. (@bvaughn en #17947)

Aunque los mensajes de los commits en inglés están escritos en imperativo, la traducción creo que se beneficiaría del infinitivo para una mayor naturalidad.

@FranciscoImanolSuarez
Copy link
Author

Ow mucho Feedback, mañana empezare a corregir lo que me pides, disculpa que no atendi antes estaba estudiando para un Examen, pero mañana ya empiezo a corregir ! Muchas gracias por tu FeedBack!

@gaearon gaearon deleted the branch reactjs:master August 5, 2021 17:46
@gaearon gaearon closed this Aug 5, 2021
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