-
Notifications
You must be signed in to change notification settings - Fork 293
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
Adding translation to react-dom: APIs #664
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@s0alken ¡Muchas gracias por tu contribución! Está casi lista para que mezclarla, solo te dejé par de comentarios para que los revises. ¡Gracias!
@@ -18,7 +18,7 @@ interface ExpandableCalloutProps { | |||
|
|||
const variantMap = { | |||
deprecated: { | |||
title: 'Deprecated', | |||
title: 'Obsoleto', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Por favor, ¿podrías revertir este cambio?
La recomendación del equipo de React es no traducir elementos de la UI de momento para evitar conflictos en el futuro:
Keep in mind translating UI is best done closer to the end. We plan to make non-trivial code changes to the repo over the next few months. So it's best to focus on the content rather than on UI components.
|
||
<Deprecated> | ||
|
||
These APIs will be removed in a future major version of React. | ||
Las siguientes APIs se eliminarán en las próximas versiones importantes de React. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Para mantener la consistencia en todas las páginas estamos utilizando la terminología mayor.menor.parche.
Las siguientes APIs se eliminarán en las próximas versiones importantes de React. | |
Las siguientes APIs se eliminarán en una próxima versión mayor de React. |
Apliqué los cambios @carburo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
¡Gracias!
Size changes📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
Traducción de react-dom: APIs. Atenderé cualquier revisión lo antes posible.