Sandbox::https://github.com/javascript-tutorial/it.javascript.info/pull/203

pierangelomiceli · 2021-02-06T00:16:04Z

No description provided.

longo-andrea · 2021-02-06T14:56:53Z

@pierangelomiceli intanto correggo l'articolo, manca però la traduzione degli esercizi e delle soluzioni

longo-andrea

ho fatto solamente un check dei typos, dopo leggo l'articolo e ti do un feedback più dettagliato

Co-authored-by: Andrea <45577511+longo-andrea@users.noreply.github.com>

longo-andrea · 2021-02-06T20:27:30Z

+Chiarimenti:
+1. Il metodo `elem.closest(selector)` ritorna l'antenato più vicino che combacia con il selettore. Nel nostro caso cerchiamo un `<td>` verso l'alto dall'elemento di origine dell'evento.
+2. Se `event.target` non è dentro nessun `<td>`, la chiamata esce immediatamente, dal momento che non c'è nulla da fare.
+3. Ne caso di tabelle annidate, `event.target` potrebbe essere un `<td>`, ma fuori dalla tabelle corrente. Quindi andiamo a controllare se sia il `<td>` della *nostra tabella*.


Suggested change

3. Ne caso di tabelle annidate, `event.target` potrebbe essere un `<td>`, ma fuori dalla tabelle corrente. Quindi andiamo a controllare se sia il `<td>` della *nostra tabella*.

3. Nel caso di tabelle annidate, `event.target` potrebbe essere un `<td>`, ma fuori dalla tabelle corrente. Quindi andiamo a controllare se sia il `<td>` della *nostra tabella*.

javascript-translate-bot · 2021-02-06T20:42:25Z

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

Co-authored-by: Andrea <45577511+longo-andrea@users.noreply.github.com>

longo-andrea · 2021-02-07T13:45:18Z

Gli esercizi tradotti sono nella PR #205

pasor1

Qualche pioccola nota

Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>

javascript-translate-bot · 2021-02-09T19:41:41Z

Thank you 💖 I updated the Progress Issue #1 🎉 🎉 🎉

traduzione event delegation

4360a5d

javascript-translate-bot added the review needed label Feb 6, 2021

javascript-translate-bot requested a review from a team February 6, 2021 00:16

javascript-translate-bot mentioned this pull request Feb 6, 2021

Italian Translation Progress #1

Open

longo-andrea reviewed Feb 6, 2021

View reviewed changes

Comment thread 2-ui/2-events/03-event-delegation/article.md Outdated

Update 2-ui/2-events/03-event-delegation/article.md

885c818

Co-authored-by: Andrea <45577511+longo-andrea@users.noreply.github.com>

longo-andrea requested changes Feb 6, 2021

View reviewed changes

javascript-translate-bot removed the review needed label Feb 6, 2021

javascript-translate-bot added the changes requested label Feb 6, 2021

pierangelomiceli and others added 5 commits February 6, 2021 23:07

Update 2-ui/2-events/03-event-delegation/article.md

b3998d6

Co-authored-by: Andrea <45577511+longo-andrea@users.noreply.github.com>

Update 2-ui/2-events/03-event-delegation/article.md

e16545e

Co-authored-by: Andrea <45577511+longo-andrea@users.noreply.github.com>

Update 2-ui/2-events/03-event-delegation/article.md

631f802

Co-authored-by: Andrea <45577511+longo-andrea@users.noreply.github.com>

Update 2-ui/2-events/03-event-delegation/article.md

5ef4eb4

Co-authored-by: Andrea <45577511+longo-andrea@users.noreply.github.com>

Update 2-ui/2-events/03-event-delegation/article.md

056feff

Co-authored-by: Andrea <45577511+longo-andrea@users.noreply.github.com>

longo-andrea added needs +1 and removed changes requested labels Feb 7, 2021

longo-andrea mentioned this pull request Feb 7, 2021

Event delegation - task and solutions #205

Merged

pasor1 approved these changes Feb 9, 2021

View reviewed changes

pierangelomiceli and others added 7 commits February 9, 2021 20:08

Update 2-ui/2-events/03-event-delegation/article.md

79052f2

Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>

Update 2-ui/2-events/03-event-delegation/article.md

be65476

Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>

Update 2-ui/2-events/03-event-delegation/article.md

86b7b35

Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>

Update 2-ui/2-events/03-event-delegation/article.md

fc8c16b

Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>

Update 2-ui/2-events/03-event-delegation/article.md

4861f1a

Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>

Update 2-ui/2-events/03-event-delegation/article.md

7fa0157

Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>

Update 2-ui/2-events/03-event-delegation/article.md

bdcf7d6

Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>

longo-andrea approved these changes Feb 9, 2021

View reviewed changes

longo-andrea merged commit 8aeeedb into javascript-tutorial:master Feb 9, 2021

	3. Ne caso di tabelle annidate, `event.target` potrebbe essere un `<td>`, ma fuori dalla tabelle corrente. Quindi andiamo a controllare se sia il `<td>` della nostra tabella.
	3. Nel caso di tabelle annidate, `event.target` potrebbe essere un `<td>`, ma fuori dalla tabelle corrente. Quindi andiamo a controllare se sia il `<td>` della nostra tabella.

Conversation

pierangelomiceli commented Feb 6, 2021

Uh oh!

longo-andrea commented Feb 6, 2021

Uh oh!

longo-andrea left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

longo-andrea Feb 6, 2021

Choose a reason for hiding this comment

Uh oh!

pierangelomiceli Feb 6, 2021

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

javascript-translate-bot commented Feb 6, 2021

Uh oh!

longo-andrea commented Feb 7, 2021

Uh oh!

pasor1 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

javascript-translate-bot commented Feb 9, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants