Visitar URL original
Moving the mouse: mouseover/out, mouseenter/leave by pierangelomiceli · Pull Request #232 · javascript-tutorial/it.javascript.info · GitHub
Skip to content

Moving the mouse: mouseover/out, mouseenter/leave - #232

Closed
pierangelomiceli wants to merge 39 commits into
javascript-tutorial:masterfrom
pierangelomiceli:016_movingmouse
Closed

Moving the mouse: mouseover/out, mouseenter/leave#232
pierangelomiceli wants to merge 39 commits into
javascript-tutorial:masterfrom
pierangelomiceli:016_movingmouse

Conversation

@pierangelomiceli

@pierangelomiceli pierangelomiceli commented Feb 24, 2021

Copy link
Copy Markdown
Contributor

No description provided.

@javascript-translate-bot
javascript-translate-bot requested a review from a team February 24, 2021 01:42
@javascript-translate-bot javascript-translate-bot mentioned this pull request Feb 24, 2021
Open
longo-andrea
longo-andrea requested changes Feb 26, 2021
View reviewed changes

@longo-andrea longo-andrea left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@pierangelomiceli ti ho lasciato un paio di commenti, parecchi sono riguardo il termine "il tooltip" che io sono abituato a chiamare "la tooltip", comunque parliamone, in caso puoi ignorarli... un'altra cosa che manca è la traduzione dei commenti nel file script.js.. il resto tutto okay 👍

Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/article.md Outdated
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/article.md Outdated
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/article.md Outdated
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/article.md Outdated
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/article.md Outdated
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/task.md
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/task.md
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/task.md
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/task.md
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/task.md
@javascript-translate-bot

javascript-translate-bot commented Feb 26, 2021

Copy link
Copy Markdown

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

pierangelomiceli and others added 9 commits February 26, 2021 22:49
@pierangelomiceli
traduzioni residue
d65aec0
@pierangelomiceli @longo-andrea
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
28d5657 
…-mouseleave/article.md

Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
7ee3383 
…-mouseleave/article.md

Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
3b43de9 
…-mouseleave/article.md

Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
98ffe81 
…-mouseleave/article.md

Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
3894b47 
…-mouseleave/article.md

Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
3a268d1 
…-mouseleave/article.md

Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
d1efa83 
…-mouseleave/article.md

Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
5831974 
…-mouseleave/article.md

Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
@pierangelomiceli @longo-andrea
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
c48bc3e 
…-mouseleave/article.md

Co-authored-by: Andrea  <45577511+longo-andrea@users.noreply.github.com>
pasor1
pasor1 approved these changes Mar 1, 2021
View reviewed changes

@pasor1 pasor1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Un po' di correzioni, da rompiballe quale sono :-D

Comment thread ...ousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/solution.view/hoverIntent.js Outdated
Comment thread ...ousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/solution.view/hoverIntent.js Outdated
Comment thread ...ousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/solution.view/hoverIntent.js Outdated
// if it's less than sensivity, then the speed is slow
// dopo ogni mousemove controlliamo la distanza
// tra le coordinate attuali e le precedenti
// se e' minore della "sensibilita'", allora la velocita' sara' bassa

@pasor1 pasor1 Mar 1, 2021

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
// se e' minore della "sensibilita'", allora la velocita' sara' bassa
// se e' minore della "sensibilità", allora la velocità sarà bassa

Comment thread ...ousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/solution.view/hoverIntent.js Outdated
Comment thread ...ousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/solution.view/hoverIntent.js Outdated

if (!this.lastTime || this.lastTime == this.prevTime) {
// cursor didn't move
// il cursore non si e' mosso

@pasor1 pasor1 Mar 1, 2021

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
// il cursore non si e' mosso
// il cursore non si è mosso

Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/article.md Outdated
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/article.md Outdated
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/article.md Outdated
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/article.md Outdated
Comment thread 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/article.md Outdated
pasor1 added 15 commits March 2, 2021 18:46
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
35f5df3 
…-mouseleave/2-hoverintent/solution.view/hoverIntent.js
@pasor1
Update hoverIntent.js
67e790d
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
cdb9182 
…-mouseleave/2-hoverintent/source.view/hoverIntent.js
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
e1e0ce1 
…-mouseleave/2-hoverintent/source.view/hoverIntent.js
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
3407a60 
…-mouseleave/2-hoverintent/source.view/hoverIntent.js
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
4eea7d4 
…-mouseleave/2-hoverintent/task.md
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
ba8ee30 
…-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
5daca2d 
…-mouseleave/article.md
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
276d01a 
…-mouseleave/article.md
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
bb18984 
…-mouseleave/article.md
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
c73d129 
…-mouseleave/article.md
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
a2848e8 
…-mouseleave/article.md
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
088e747 
…-mouseleave/article.md
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
a092649 
…-mouseleave/article.md
@pasor1
Update 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter…
0ce6c7c 
…-mouseleave/article.md
@pasor1 pasor1 added the invalid This doesn't seem right label Mar 2, 2021
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

invalid This doesn't seem right needs +1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@pierangelomiceli @javascript-translate-bot @longo-andrea @pasor1