Skip to content

Searching: getElement*, querySelector* #311

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

Merged

Conversation

elemarmar
Copy link
Contributor

¡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 🦊

@CLAassistant
Copy link

CLAassistant commented Jul 20, 2020

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

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 July 20, 2020 14:30
…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

¡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
Member

¡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

@joaquinelio Aprobamos? A mi me gusta!

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.

@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
Member

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
Member

@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 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

Choose a reason for hiding this comment

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

Arreglamos esto, y listo!

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

Pero qué :jovér.

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

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

/done

@javascript-translate-bot javascript-translate-bot requested a review from a team July 21, 2020 06:15
@joaquinelio joaquinelio merged commit 35d7d53 into javascript-tutorial:master Jul 21, 2020
@elemarmar elemarmar deleted the search-elements-dom-emm branch July 21, 2020 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
6 participants