javascript-tutorial/es.javascript.infoPublic

NotificationsYou must be signed in to change notification settings
Fork230
Star600

Popups and window methods#359

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 36 commits intojavascript-tutorial:masterfromDerlys:derlys/popups-windows

Sep 8, 2020

Merged

Popups and window methods#359

joaquinelio merged 36 commits intojavascript-tutorial:masterfromDerlys:derlys/popups-windows

Sep 8, 2020

Conversation

Copy link

Contributor

Derlys commentedAug 25, 2020

No description provided.

javascript-translate-bot added the review needed label

Aug 25, 2020

Copy link

CLAassistant commentedAug 25, 2020•
edited
Loading

All committers have signed the CLA.

Update 3-frames-and-windows/01-popup-windows/article.md

50534a2

Derlys force-pushed thederlys/popups-windows branch fromed73617 to50534a2Compare

August 25, 2020 22:10

joaquinelio changed the title~~Update 3-frames-and-windows/01-popup-windows/article.md~~Popups and window methods

Aug 25, 2020

joaquinelio self-requested a review

August 25, 2020 22:24

joaquinelio added7 commits

August 25, 2020 19:32

numeración

a3af98b

numeracion

4d63ccd

numeracion

ccf21bd

restablece codigo original lin 141

f8737a1

numeracion

9de22f2

linea de más, numeracion

be5c599

numeracion

337d553

joaquinelio requested changes

Aug 26, 2020

View reviewed changes

Copy link

Member

joaquinelio left a comment

Choose a reason for hiding this comment

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

Uf que arduo fue esto
espero qie no te desanime =)

Conviene que el nombre del PR sea el del articulo, asi Don Bot lo registra. Igual anda perezoso y lo terminamos agregando a mano asi que no big deal.

Cuida los numeros de linea, facilita la review y ESPECIALMENTE los merge de actualizaciones del inglés.
Me atreví a corregirlos para esta review.
No agregues/quites lineas.

Tampoco deberias cambiar el estilo, veo que esspecialmente cambiaste comillas y pusiste esspacios, (y en ningun proyecto ajeno) aunque te parezca horrible ( salvo si es bug, que mandamos pr al inglés)
Por ej nunca uso ";" en mi codigo pero si es compartido me adapto a poner esa porquería inutil.

No modifiques codigo si no podes probarlo plis
Si te interesa podes instalar el server local para probar,
ademas asi se entiende como funcionan los markups

3-frames-and-windows/01-popup-windows/article.md OutdatedShow resolvedHide resolved

javascript-translate-bot removed the review needed label

Aug 26, 2020

Copy link

javascript-translate-bot commentedAug 26, 2020

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

javascript-translate-bot added the changes requested label

Aug 26, 2020

iliakan mentioned this pull request

Aug 26, 2020

Spanish Translation Progress#17

Open

EzequielCasteand others added12 commits

August 28, 2020 12:50

Update 3-frames-and-windows/01-popup-windows/article.md

338de5a

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

5fa8006

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

0c9c5b3

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

c91d027

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

2f885e5

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

c56b3de

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

897ed64

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

469f90d

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

d60fe91

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

107fc6f

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

1bbb05c

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

5b2693f

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

joaquinelio added2 commits

September 2, 2020 03:30

numeracion 219

17680d8

numeracion 244

a54f479

joaquinelio requested changes

Sep 2, 2020

View reviewed changes

Copy link

Member

joaquinelio left a comment

Choose a reason for hiding this comment

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

AAAHHHH!!!
Perdonperdonperdon ¡Me pasé por alto unos cambios importantes!
Muchos atributos que son parametros, no se traducen `width:200,height=100
Va de nuevo...

No corrijo formatos porque el original tampoco es homogeneo
además veo que el también empezó a agregar los ";"

3-frames-and-windows/01-popup-windows/article.md OutdatedShow resolvedHide resolved

Copy link

javascript-translate-bot commentedSep 2, 2020

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

Copy link

Member

joaquinelio commentedSep 2, 2020

Reto:
Lo es. Fue tambien mi primera vez. y aprendi un monton de git y js en pocos meses así que te entiendo.

PR:
Nada importante.
Se debería poner el nombre exacto del articulo como nombre de PR, pero igual el bot a veces lo ignora y lo terminamos agregando a mano.

Comillas:
SEGURAMENTE a tu editor se le puede configurar el corrector de estilo o deshabilitarlo.
Es importante porque podriass trabajar en diferentes equipos con diferentes estilos y deberias respetarlos aunque sean una porquería.
Y acá especificamente porque mucho del codigo puede tener markup, parece que tu editor lo borró.

De nuevo al ruedo... 👍

Derlysand others added10 commits

September 2, 2020 13:16

Update 3-frames-and-windows/01-popup-windows/article.md

1e3f2ca

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

2b786f0

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

46f65c7

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

d915a5c

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

6d3080c

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

60b8418

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

91874b4

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

47dca55

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

be7c8f3

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 3-frames-and-windows/01-popup-windows/article.md

cabcd42

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

joaquinelio approved these changes

Sep 4, 2020

View reviewed changes

javascript-translate-bot added needs +1 and removed changes requested labels

Sep 4, 2020

Derlys closed this

Sep 4, 2020

Copy link

Member

joaquinelio commentedSep 4, 2020

@Derlys
Me llegó el mensaje "closed"... je quiero creer que fue un dedo mal puesto, ¿no?
o será que te arrepentiste...
falta un reviewer y va para el sitio

Copy link

ContributorAuthor

Derlys commentedSep 4, 2020 via email

Hola, es que lo hice mal, cual es el paso a seguir?El vie., 4 sept. 2020 a las 15:18, joaquinelio (<notifications@github.com>)escribió:

…

@Derlys <https://github.com/Derlys> Me llegó el mensaje "closed"... je quiero creer que fue un dedo mal puesto, ¿no? o será que te arrepentiste... falta un reviewer y va para el sitio — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#359 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AN6GO75KB6II5EXBWEYANELSEFDQFANCNFSM4QLD7G4A> .

Copy link

Member

joaquinelio commentedSep 4, 2020

Si está mal, se puede cerrar y hacer uno nuevo
o mejor corregirlo...

¿qué es lo que está mal?

Derlys reopened this

Sep 5, 2020

Copy link

ContributorAuthor

Derlys commentedSep 5, 2020

Lo que te quiero decir es que no se que se debe hacer ahora con esto, dice changes aprobado ahora que sigue?

Copy link

Member

joaquinelio commentedSep 5, 2020

Ah, nada de tu lado hasta que alguien más lo vea.

el +1 significa otro reviewer, una segunda revisión...
andamos escasos de reviewers asi que solo hay que esperar.

vplentinax approved these changes

Sep 8, 2020

View reviewed changes

Copy link

Contributor

vplentinax left a comment

Choose a reason for hiding this comment

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

Hay algunas palabras que pueden cambiarse para no romper el flujo de lectura pero se puede arreglar en alguna revisión posterior de archivos traducidos.