-
Notifications
You must be signed in to change notification settings - Fork 226
Object methods, "this" #343
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
Conversation
1-js/04-object-basics/04-object-methods/4-object-property-this/task.md
Outdated
Show resolved
Hide resolved
Please make the requested changes. After it, add a comment "/done". |
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
…/task.md Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
/done |
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.
Hay algo que me causa confusión y creo que es bueno que nos lo verifiquemos. Leí un diccionario y creo que tienes razón en el "before" pero no estoy seguro ya que hay código de ejemplo que no puedo observar en modo review, así que revisaré el artículo con formato. Por otro lado a veces es difícil concentrarse en todo el contexto del artículo mientras haces review, entonces creo que eso me está pasando. Lo leeré y comprenderé antes de regresar a aprobar en caso de que tengas razón
1-js/04-object-basics/04-object-methods/4-object-property-this/solution.md
Outdated
Show resolved
Hide resolved
1-js/04-object-basics/04-object-methods/4-object-property-this/solution.md
Outdated
Show resolved
Hide resolved
|
||
The value of `this` is one for the whole function, code blocks and object literals do not affect it. | ||
El valor de `this` es uno para la función entera, bloques de código y objetos literales no lo afectan. |
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.
El valor de `this` es uno para la función entera, bloques de código y objetos literales no lo afectan. | |
El valor de `this` es uno para la función entera. Bloques de código y objetos literales no lo afectan. |
Funciona bien así como dos oraciones independientes en el mismo párrafo y pienso que se entiende mejor. Si lo leo con la coma me voy de corrido y pienso que es "función entera, bloques de código y objetos literales" hasta que mi cerebro hace corto circuito al encontrarse con "no lo afectan".
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.
@maksumi la coma es justamente de pausa en este caso. No se recomiendo usa punto para dar pausa o separar oraciones pequeñas.
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.
creo qu amerita un " ; " !!!
si está mal no me importa.
Creo que se lee bien en una oracion si lee con el ritmo adecuado,
tambien reo que se puede poner en dos oraciones aunque se corte un poquito
pero asi no hay duda:
El valor de this
es uno para la función entera; bloques de código y objetos literales no lo afectan.
podria haber puesto
fun entera, ni bloquess ni lieterales...
fun entera (biques y leterales....)
muy feo.
1-js/04-object-basics/04-object-methods/4-object-property-this/solution.md
Outdated
Show resolved
Hide resolved
…/solution.md Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
el nombre viejo es usado! a.X.b No deja uar batch aca "onversation" , hice todo en "files changed" y otrass resolvi |
/done |
Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉 |
No description provided.