Skip to content

Conversation

@joaquinelio
Copy link
Member

@joaquinelio joaquinelio commented Oct 11, 2020

https://es.javascript.info/closure ultimo task

Editado sobre gitlens
deberia estar identico...

sobra un svg ahora

@vplentinax
Copy link
Contributor

Se añadieron muchos espacios, ¿eso es correcto?

@joaquinelio
Copy link
Member Author

joaquinelio commented Oct 15, 2020

Sí.

No entiendo como funciona el espaciado, a veces SERVER ignora el indentado... pero en este caso, no se como pero acomoda la columna dentro de la caja

Los err que cometí en la ultima sync fueron por no haber visto bien la numeracion,
tambien cometemos muchos err de espaciado ( y es imposible detectarlo en las review online de retraducciones)
por eso ahora edito DIRECTAMENTE sobre la comparación gitlens, ahí me di cuenta del espaciado.
image
Las lineas verdes son editables, es muy facil corregir numeracion primero y el texto despues.

dif de indentado ademas un err de formato. una comita que falte y los anidados se van al *
image

@joaquinelio
Copy link
Member Author

Ahora veo que también se agregaron espacios en el código... espacios que no respetan estilo pero están en el original.
No fue Ilya el que hizo la mod de código. Pero lo aprobó... (y rescribio algo de la explicacion)

No es importante en ESTE codigo, solo comento.

Copy link
Contributor

@vplentinax vplentinax left a comment

Choose a reason for hiding this comment

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

ok

@joaquinelio joaquinelio merged commit 3dd9041 into javascript-tutorial:master Oct 21, 2020
@joaquinelio joaquinelio deleted the pum branch October 21, 2020 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants