javascript-tutorial/es.javascript.infoPublic

NotificationsYou must be signed in to change notification settings
Fork230
Star600

Arrays#309

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

joaquinelio merged 28 commits intojavascript-tutorial:masterfromjoaquinelio:arr

Jul 22, 2020

Merged

Arrays#309

joaquinelio merged 28 commits intojavascript-tutorial:masterfromjoaquinelio:arr

Jul 22, 2020

Conversation

Copy link

Member

joaquinelio commentedJul 19, 2020

Usé "corchetes" para "square brackets", como aprendí en el colegio ¿es lo usual?

Arrays

042d43f

javascript-translate-bot added the review needed label

Jul 19, 2020

javascript-translate-bot requested a review froma team

July 19, 2020 22:55

iliakan mentioned this pull request

Jul 19, 2020

Spanish Translation Progress#17

Open

Copy link

Contributor

vplentinax commentedJul 20, 2020

Si, corchetes!

maksumi approved these changes

Jul 21, 2020

View reviewed changes

Copy link

Collaborator

maksumi left a comment

Choose a reason for hiding this comment

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

Algunos typos y algunas sugerencias conceptuales. Na'más...

1-js/05-data-types/04-array/10-maximal-subarray/solution.md OutdatedShow resolvedHide resolved

1-js/05-data-types/04-array/10-maximal-subarray/solution.mdShow resolvedHide resolved

1-js/05-data-types/04-array/10-maximal-subarray/solution.md OutdatedShow resolvedHide resolved

1-js/05-data-types/04-array/article.md OutdatedShow resolvedHide resolved

javascript-translate-bot added needs +1 and removed review needed labels

Jul 21, 2020

Copy link

Contributor

vplentinax commentedJul 21, 2020•
edited
Loading

Algunos typos y algunas sugerencias conceptuales. Na'más...

@maksumi todo lo que puedas corregir o sugerir, incluido typos, añadelos o comenta la línea para comparar.

Edición:

Lo vi desde el cell y no me di cuenta que habías hecho las sugerencias! Sorry!

joaquinelioand others added21 commits

July 21, 2020 06:09

Update 1-js/05-data-types/04-array/10-maximal-subarray/solution.md

5314f8f

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/10-maximal-subarray/solution.md

b3a0cc0

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/10-maximal-subarray/solution.md

af2e2a0

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/5-array-input-sum/task.md

a373976

Maldito teclado.  La s me está fallando.Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

baa16d7

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

71793c9

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

1a517fd

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

a73a204

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

e88e22a

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/5-array-input-sum/task.md

fef7302

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/2-create-array/task.md

93aedf5

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/3-call-array-this/solution.md

cc3750c

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/3-call-array-this/task.md

c2f4f67

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

05a0618

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/5-array-input-sum/solution.md

383249d

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

f122d1e

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

8fdb93a

ocioso...pero me da igualCo-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

726df56

AAAAHHHHHHH!!!!Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

b3e4585

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

2f951b1

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

785ff52

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

joaquinelioand others added4 commits

July 21, 2020 08:59

Update 1-js/05-data-types/04-array/article.md

fe7c1bf

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

214685f

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update 1-js/05-data-types/04-array/article.md

db4e7fb

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Update article.md

778666a

Copy link

MemberAuthor

joaquinelio commentedJul 21, 2020

/done
pero no del todo, un par de suggest aun por resolver@maksumi @vplentinax

De acuerdo, longitud y largo es lo mismo pero longitud se lee más claro y cómodo.
si hubiera sabido que eran tantos habria editado con /reemplazar.
LISTO lo hice, encontre 2 más. irrelevantess, pero para concordar.

ingreasar por entrar, ok, se me escapo

¿Te molesta la voz pasiva?
La evito si está de más pero a veces ayuda en la secuencia de lo que se expresa.
Aca me daba igual.

Te desprecié dos o tres, alguno para debate.
habrás de leer...
1 5 4 10 solution 40, 66

1 5 4 article 124 136 190 editadas sin el suggest,
lo que muestra la conversación no es lo que quedó.

¿Deberia poner mayusc y punto final en los // ? nah. Pero me da nosequé.

monton de commits, debi usar modo batch.

javascript-translate-bot added the review needed label

Jul 21, 2020

maksumi reviewed

Jul 21, 2020

View reviewed changes

1-js/05-data-types/04-array/article.md OutdatedShow resolvedHide resolved

Update 1-js/05-data-types/04-array/article.md

bcb2dc4

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Copy link

MemberAuthor

joaquinelio commentedJul 22, 2020

@vplentinax ,pido tu opinión españolísima (aunque seas implantada)
1 5 4 10 solution, 2 ssuggest que desprecié.
Uno es la coma, no me importa quitarla pero creo que va.
Segundo, el uso de "actual" para ssignificar lo que esttá en curso en lugar del que creí más ajustado "corriente", ( y ya está explicando código, por eso me detengo a pensarlo)
pero noté que en otro lado también usaron "actual"... Casi preferiría "elemento presente".
¿te parece que lo cambie a "actual"?

Copy link

Contributor

vplentinax commentedJul 22, 2020

@vplentinax ,pido tu opinión españolísima (aunque seas implantada)
1 5 4 10 solution, 2 ssuggest que desprecié.
Uno es la coma, no me importa quitarla pero creo que va.
Segundo, el uso de "actual" para ssignificar lo que esttá en curso en lugar del que creí más ajustado "corriente", ( y ya está explicando código, por eso me detengo a pensarlo)
pero noté que en otro lado también usaron "actual"... Casi preferiría "elemento presente".
¿te parece que lo cambie a "actual"?

Hecho!

maksumi approved these changes

Jul 22, 2020

View reviewed changes

Copy link

Collaborator

maksumi left a comment

Choose a reason for hiding this comment

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

Ya aprové... Lo dejo a tu criterio porque la traducción está muy bien y solo son sugerencias alternativas que a mi criterio creo que funcionarían también

Update 1-js/05-data-types/04-array/10-maximal-subarray/solution.md

6a6b3f8

NO creo que sea mejor "actual".  Pero busqué en el repo, y fue lo usado.  Sea por correcto o por convención, pensando en quien aprende, prefiero la homogeneidad.  Va ACTUAL.Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

Copy link

MemberAuthor