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

Searching: getElement*, querySelector* #311

Merged

Conversation

@elemarmar
Copy link
Contributor

@elemarmar elemarmar commented Jul 20, 2020

¡Hola! Es mi primera contribución así que quizá haya "patinado un poco". Cualquier feedback es bien recibido y ¡espero aprender mucho para poder seguir contribuyendo!
Un saludo desde España 🦊

elemarmar added 12 commits Jul 18, 2020
@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Jul 20, 2020
@CLAassistant
Copy link

@CLAassistant CLAassistant commented Jul 20, 2020

CLA assistant check
All committers have signed the CLA.

@iliakan iliakan mentioned this pull request Jul 20, 2020
159 of 174 tasks complete
Copy link
Contributor

@joaquinelio joaquinelio left a comment

Buena adición.
Todos patinamos, y mucho. Por eso la doble revisión. Que también pueden equivocarse.

Al segundo reviewer, el título es ok pero el bot no lo reconoció, (se habrá confundido con los *)
el merge habrá que agregarlo al issue a mano.

@javascript-translate-bot

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

elemarmar and others added 4 commits Jul 20, 2020
…tion.md

Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@elemarmar
Copy link
Contributor Author

@elemarmar elemarmar commented Jul 20, 2020

¡Muchas gracias por la revisión! He visto que algunos errores eran muy tontos de no haber yo revisado bien. Lo haré mejor la próxima vez para ahorraros el trabajo. ¡Trabajazo el vuestro! 👏🏻

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

@joaquinelio joaquinelio commented Jul 20, 2020

¡Muchas gracias por la revisión! He visto que algunos errores eran muy tontos de no haber yo revisado bien. Lo haré mejor la próxima vez para ahorraros el trabajo. ¡Trabajazo el vuestro! 👏🏻

Por eso el mensaje primero. Siempre se escapa algo, el cerebro nos trampea y lee lo que cree que dice.

"Lo haré mejor la próxima" a mí no me sirvió, nuestro tramposo cerebro es más inteligente y perezoso que nosotros.
Mi consejo es dejarlo reposar y releerlo al otro día, consejo que tampoco sigo.

@vplentinax
Copy link
Contributor

@vplentinax vplentinax commented Jul 21, 2020

@joaquinelio Aprobamos? A mi me gusta!

Copy link
Contributor

@vplentinax vplentinax left a comment

@joaquinelio, @elemarmar, como opinión personal, no veo la necesidad de aclarar el término "hover". Dentro del mundo del desarrollo creo que ya muchos lo tenemos asumido como el evento del mouse sobre un elemento. Pero, como ustedes crean, me parecerá bien, quizás sea neceario para los que desconocen HTML o CSS.

@joaquinelio
Copy link
Contributor

@joaquinelio joaquinelio commented Jul 21, 2020

@vplentinax @elemarmar,

"Dentro del mundo del desarrollo creo que ya muchos lo tenemos asumido"
"quizás sea neceario para los que desconocen HTML o CSS."

Apunto a ese que desconoce HTM, CSS... y JS. ¡Es justamente la puerta de entrada!
Es una discusión que debimos tener supongo, a quién le hablamos cuando hablamos JS español.

I have a dream.

Apunto a ese pibe curioso, inquieto, que le gusta el desafío y que todo lo nuevo lo aprenda en este sitio, que sea capaz de entenderlo y aprenderlo aunque le sea arduo o porque es un desafío.

Me gusta poner el ejemplo del monje medieval que aprende a leer de adulto, y en voz alta para entender lo que sale de sus labios. De cómo se reiría ante la idea de un niño de 4 años leyendo mejor que él.
No apunto al de 4 pero sí al joven mucho antes de que pueda pensar en una carrera universitaria.

Y sin infantilizar, claro. Mantener el esstilo Ilya. Un traductor no debe reinterpretar estilo.

Jaja qué volado. No va a cambiar porque aclaren o no hover, como quieran. Me gustó escribirlo anque.

@joaquinelio
Copy link
Contributor

@joaquinelio joaquinelio commented Jul 21, 2020

@joaquinelio Aprobamos? A mi me gusta!

Sí, anda bien, es fuido. Solo no me gusta ese "flota o sobrevuela" jajajaja quien lo habrá sugerido...
Peleénme, a veces divago pero al rato los coágulos decantan y respondo.

Pero ALGO pondría en la primera referencia de Hover = ...

Pero sí, pa'l merge.
incluso si no cambia esas cositas que no hacen al JS.

¿asi que viviste en Epaña? Entonces no te parecio raro el "por diferencia".
"por varios cuerpos" diria mi tio apostador.

@vplentinax
Copy link
Contributor

@vplentinax vplentinax commented Jul 21, 2020

@joaquinelio

¿asi que viviste en Epaña? Entonces no te parecio raro el "por diferencia".
"por varios cuerpos" diria mi tio apostador.

Ajam, no me pareció raro, rara me pareció tu sugerencia "y mucho" jajajjaja...

Por cierto, @joaquinelio aprueba para que aparezca tu check!

Copy link
Contributor

@vplentinax vplentinax left a comment

Arreglamos esto, y listo!

2-ui/1-document/04-searching-elements-dom/article.md Outdated Show resolved Hide resolved
Copy link
Contributor

@joaquinelio joaquinelio left a comment

Pero qué :jovér.

elemarmar and others added 2 commits Jul 21, 2020
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
@elemarmar
Copy link
Contributor Author

@elemarmar elemarmar commented Jul 21, 2020

Me lo pasé genial leyendo su conversación, haha. Creo que ahora es cuando le pongo lo del done. A ver qué taal....

@elemarmar
Copy link
Contributor Author

@elemarmar elemarmar commented Jul 21, 2020

/done

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Jul 21, 2020
@joaquinelio joaquinelio merged commit 35d7d53 into javascript-tutorial:master Jul 21, 2020
1 check passed
1 check passed
license/cla Contributor License Agreement is signed.
Details
@elemarmar elemarmar deleted the elemarmar:search-elements-dom-emm branch Jul 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.