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

Translate useLayoutEffect #626

Merged
merged 25 commits into from
Mar 28, 2023
Merged

Translate useLayoutEffect #626

merged 25 commits into from
Mar 28, 2023

Conversation

luiserdef
Copy link
Contributor

@github-actions
Copy link

github-actions bot commented Feb 7, 2023

Size Changes

📦 Next.js Bundle Analysis

This analysis was generated by the next.js bundle analysis action 🤖

This PR introduced no changes to the javascript bundle 🙌

beta/src/content/reference/react/useLayoutEffect.md Outdated Show resolved Hide resolved

* When Strict Mode is on, React will **run one extra development-only setup+cleanup cycle** before the first real setup. This is a stress-test that ensures that your cleanup logic "mirrors" your setup logic and that it stops or undoes whatever the setup is doing. If this causes a problem, [you need to implement the cleanup function.](/learn/synchronizing-with-effects#how-to-handle-the-effect-firing-twice-in-development)
* Cuando el modo estricto está activado, React va a **ejecutar un setup adicional solo para desarrollo + ciclo de limpieza** antes del primer setup real. Esta es una prueba de estrés que asegura que tu lógica de limpieza sea un "espejo" de tu lógica del setup y se detenga o se deshaga lo que sea que tu setup esté haciendo. Si esto causa un problema [necesitas implementar la función de limpieza.](/learn/synchronizing-with-effects#how-to-handle-the-effect-firing-twice-in-development)
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
* Cuando el modo estricto está activado, React va a **ejecutar un setup adicional solo para desarrollo + ciclo de limpieza** antes del primer setup real. Esta es una prueba de estrés que asegura que tu lógica de limpieza sea un "espejo" de tu lógica del setup y se detenga o se deshaga lo que sea que tu setup esté haciendo. Si esto causa un problema [necesitas implementar la función de limpieza.](/learn/synchronizing-with-effects#how-to-handle-the-effect-firing-twice-in-development)
* Cuando el Modo Estricto está activado, React va a **ejecutar solo en desarrollo un ciclo adicional de setup+limpieza** antes del primer setup real. Esta es una prueba de estrés que asegura que tu lógica de limpieza sea un "espejo" de tu lógica del setup y se detenga o se deshaga lo que sea que tu setup esté haciendo. Si esto causa un problema [necesitas implementar la función de limpieza.](/learn/synchronizing-with-effects#how-to-handle-the-effect-firing-twice-in-development)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Corregí esta parte en el último commit (habia un espacio extra que no debe estar después de ciclo de limpieza** )

beta/src/content/reference/react/useLayoutEffect.md Outdated Show resolved Hide resolved
beta/src/content/reference/react/useLayoutEffect.md Outdated Show resolved Hide resolved
beta/src/content/reference/react/useLayoutEffect.md Outdated Show resolved Hide resolved
beta/src/content/reference/react/useLayoutEffect.md Outdated Show resolved Hide resolved
beta/src/content/reference/react/useLayoutEffect.md Outdated Show resolved Hide resolved
beta/src/content/reference/react/useLayoutEffect.md Outdated Show resolved Hide resolved
beta/src/content/reference/react/useLayoutEffect.md Outdated Show resolved Hide resolved
beta/src/content/reference/react/useLayoutEffect.md Outdated Show resolved Hide resolved
carburo

This comment was marked as duplicate.

@carburo carburo self-requested a review March 24, 2023 15:12
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.

¡Gracias por tu contribución! He dejado algunos comentarios para que revises.

@luiserdef
Copy link
Contributor Author

Me sorprendió las faltas de ortografía que se me escaparon. Ya he realizado los cambios que me has sugerido por lo que ahora debería estar todo en orden.

@luiserdef luiserdef requested a review from carburo March 25, 2023 01:54
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.

🚀

@carburo carburo enabled auto-merge (squash) March 28, 2023 22:15
@carburo carburo merged commit d1c2dec into reactjs:main Mar 28, 2023
@luiserdef luiserdef deleted the use-layout-effect branch June 15, 2023 04:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants