Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

Already on GitHub?Sign in to your account

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 🦊

maksumi reacted with heart emoji
@CLAassistant
Copy link

CLAassistant commentedJul 20, 2020
edited
Loading

CLA assistant check
All committers have signed the CLA.

Copy link
Member

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

elemarmar reacted with heart emoji
@javascript-translate-bot

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

elemarmarand others added4 commitsJuly 20, 2020 14:30
…tion.mdCo-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
ContributorAuthor

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

elemarmar reacted with laugh emoji

@vplentinax
Copy link
Contributor

@joaquinelio Aprobamos? A mi me gusta!

Copy link
Contributor

@vplentinaxvplentinax 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 commentedJul 21, 2020
edited
Loading

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

elemarmar reacted with laugh emoji

@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 commentedJul 21, 2020
edited
Loading

@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

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

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

elemarmarand others added2 commitsJuly 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
ContributorAuthor

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
ContributorAuthor

/done

vplentinax reacted with thumbs up emoji

@javascript-translate-botjavascript-translate-bot requested a review froma teamJuly 21, 2020 06:15
@joaquineliojoaquinelio merged commit35d7d53 intojavascript-tutorial:masterJul 21, 2020
@elemarmarelemarmar deleted the search-elements-dom-emm branchJuly 21, 2020 22:31
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@maksumimaksumimaksumi left review comments

@vplentinaxvplentinaxvplentinax approved these changes

@joaquineliojoaquineliojoaquinelio approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

6 participants
@elemarmar@CLAassistant@javascript-translate-bot@joaquinelio@vplentinax@maksumi

[8]ページ先頭

©2009-2025 Movatter.jp