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.

Sign up for GitHub

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

Jump to bottom

Bubbling and capturing #198

Merged
merged 15 commits into from
Feb 1, 2021
Merged

Bubbling and capturing #198

merged 15 commits into from
Feb 1, 2021

Conversation

pierangelomiceli
Copy link
Contributor

No description provided.

@javascript-translate-bot javascript-translate-bot requested a review from a team January 30, 2021 12:48
@longo-andrea longo-andrea changed the title Bubbling e capturing Bubbling and capturing Jan 30, 2021
@longo-andrea longo-andrea mentioned this pull request Jan 30, 2021
Open
longo-andrea
longo-andrea approved these changes Jan 30, 2021
View reviewed changes
Copy link
Contributor

@longo-andrea longo-andrea left a comment

Choose a reason for hiding this comment

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

L'articolo è okay quindi lo ho approvato, ti ho lasciato giusto un paio di suggerimenti (principalmente errori di battitura)

2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
pierangelomiceli and others added 5 commits January 30, 2021 20:28
@pierangelomiceli @longo-andrea
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
f142dbe 
Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
9f40d55 
Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
30c7cae 
Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
6949aa0 
Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
0f4f01b 
Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
pasor1
pasor1 approved these changes Jan 31, 2021
View reviewed changes
Copy link
Member

@pasor1 pasor1 left a comment

Choose a reason for hiding this comment

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

Un po' di correzioni e modifiche

2-ui/2-events/02-bubbling-and-capturing/article.md Outdated

**When an event happens on an element, it first runs the handlers on it, then on its parent, then all the way up on other ancestors.**
**Quando viene innescato un evento su un elemento, come prima cosa vengono eseguito i gestori ad esso assegnati, poi ai nodi genitori, ed infine risale fino agli altri nodi antenati.**
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
**Quando viene innescato un evento su un elemento, come prima cosa vengono eseguito i gestori ad esso assegnati, poi ai nodi genitori, ed infine risale fino agli altri nodi antenati.**
**Quando viene innescato un evento su un elemento, come prima cosa vengono eseguiti i gestori ad esso assegnati, poi quelli assegnati ai nodi genitori, ed infine si risale fino a quelli assegnati ai nodi antenati.**

2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Outdated

For instance, a `focus` event does not bubble. There are other examples too, we'll meet them. But still it's an exception, rather than a rule, most events do bubble.
L'evento `focus`, per esempio, non è tra questi. Ci sono anche altri esempi, che incontreremo. Ma ancorauna volta è un'eccezione, piuttosto che una regola, in quanto la maggior parte degli eventi sono soggetti al *bubbling*.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
L'evento `focus`, per esempio, non è tra questi. Ci sono anche altri esempi, che incontreremo. Ma ancorauna volta è un'eccezione, piuttosto che una regola, in quanto la maggior parte degli eventi sono soggetti al *bubbling*.
L'evento `focus`, per esempio, non è tra questi. Ci sono anche altri esempi, che incontreremo. Ma, ancora una volta, questi sono un'eccezione piuttosto che una regola, in quanto la maggior parte degli eventi sono soggetti al *bubbling*.

2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved
pierangelomiceli and others added 9 commits January 31, 2021 12:41
@pierangelomiceli @pasor1
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
c413f44 
Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>
@pierangelomiceli @pasor1
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
c4369e9 
Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>
@pierangelomiceli @pasor1
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
c10bd5f 
Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>
@pierangelomiceli @pasor1
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
fdaf88f 
Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>
@pierangelomiceli @pasor1
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
63c6f08 
Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>
@pierangelomiceli @pasor1
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
1f11322 
Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>
@pierangelomiceli @pasor1
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
fe3fbba 
Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>
@pierangelomiceli @pasor1
Update 2-ui/2-events/02-bubbling-and-capturing/article.md
15e0a66 
Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>
@longo-andrea @pasor1
Apply suggestions from code review
6e92e44 
Co-authored-by: Simone Pasini <66781510+pasor1@users.noreply.github.com>
@longo-andrea longo-andrea merged commit 9156930 into javascript-tutorial:master Feb 1, 2021
@javascript-translate-bot
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@longo-andrea longo-andrea longo-andrea approved these changes

@pasor1 pasor1 pasor1 approved these changes

Assignees
No one assigned
Labels
needs +1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@pierangelomiceli @javascript-translate-bot @longo-andrea @pasor1