Skip to content

Commit 3581cab

Browse files
authored
Apply suggestions from code review
1 parent d7052eb commit 3581cab

File tree

4 files changed

+11
-11
lines changed
  • 2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave

4 files changed

+11
-11
lines changed

2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/1-behavior-nested-tooltip/solution.view/index.html

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -54,19 +54,19 @@
5454

5555
<p>Жила-була мама-свиня, у якої було троє поросят.</p>
5656

57-
<p>Троє поросят виросли такі великі, що їхня мати сказала їм: "Ви занадто великі, щоб жити тут далі. Ви повинні йти і будувати собі будинки. Але бережіть себе, щоб вовк вас не спіймав".</p>
57+
<p>Троє поросят виросли такі великі, що їхня мати сказала їм: "Ви занадто великі, щоб жити тут далі. Ви повинні йти і будувати собі будинки. Але бережіть себе, щоб вовк вас не спіймав."</p>
5858

5959
<p>Троє поросят рушили. "Будемо берегти один одного, щоб вовк нас не спіймав", - сказали вони.</p>
6060

61-
<p>Незабаром вони зустріли чоловіка.<a href="https://uk.wikipedia.org/wiki/Троє_поросят" data-tooltip="Читайте далі…">Наведіть на мене вказівник</a></p>
61+
<p>Незабаром вони зустріли чоловіка.<a href="https://uk.wikipedia.org/wiki/Троє_поросят" data-tooltip="Читайте далі…">Наведіть на мене мишу</a></p>
6262

6363
</div>
6464

6565
<script>
6666
let tooltip;
6767

6868
document.onmouseover = function(event) {
69-
// важливо: миша, що швидко рухається, може "стрибнути" прямо до дочірнього вузла на анотованому вузлі, пропускаючи батьківського
69+
// важливо: миша, що швидко рухається, може "стрибнути" прямо до дочірнього вузла, пропускаючи батьківський
7070
// тому mouseover може статися на дочірньому елементі.
7171

7272
let anchorElem = event.target.closest('[data-tooltip]');

2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/solution.view/hoverIntent.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,16 +32,16 @@ class HoverIntent {
3232
onMouseOver(event) {
3333

3434
if (this.isOverElement) {
35-
// якщо ми знову пройшли над елементом, ігноруємо подію
36-
// ми вже вимірюємо швидкість
35+
// якщо ми знову пройшли над елементом, ігноруємо подію,
36+
// бо ми вже вимірюємо швидкість
3737
return;
3838
}
3939

4040
this.isOverElement = true;
4141

4242
// після кожного руху миші ми будемо перевіряти відстань
4343
// між попередньою та поточною координатами миші
44-
// якщо менше ніж значення sensitivity, швидкість повільна
44+
// якщо ця відстань менше ніж значення sensitivity, швидкість повільна
4545

4646
this.prevX = event.pageX;
4747
this.prevY = event.pageY;

2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/2-hoverintent/task.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,11 @@ importance: 5
44

55
# "Розумна" підказка
66

7-
Напишіть функцію, яка показуватиме спливаючу підказку над елементом лише тоді, коли відвідувач прямую вказівник миші безпосередньо *до нього*, але не *крізь нього*.
7+
Напишіть функцію, яка показуватиме спливаючу підказку над елементом лише тоді, коли користувач веде вказівник миші безпосередньо *до нього*, але не *крізь нього*.
88

9-
Іншими словами, якщо відвідувач наводить вказівник миші на елемент і зупиняється на ньому, показується підказка. І якщо просто провели мишею, то показувати підказку взагалі не потрібно, кому може сподобатись додаткове миготіння?
9+
Іншими словами, якщо користувач наводить вказівник миші на елемент і зупиняється на ньому, показується підказка. І якщо просто провели мишею, то показувати підказку взагалі не потрібно, кому може сподобатись додаткове миготіння?
1010

11-
Технічно ми можемо виміряти швидкість я кою рухається вказівник над елементом, і якщо вона швидкість повільна, ми припускаємо, що вказівник проходить "над елементом" і показуємо спливаючу підказку, якщо вона швидка, ми ігноруємо цей прохід вказівника.
11+
Технічно ми можемо виміряти швидкість з якою рухається вказівник над елементом, і якщо вона (швидкість) повільна, ми припускаємо, що вказівник проходить "над елементом" і показуємо спливаючу підказку, якщо вона швидка, ми ігноруємо цей прохід вказівника.
1212

1313
У цьому завданні ви маєте описати клас і створити універсальний об’єкт `new HoverIntent(options)`
1414

2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ let currentElem = null;
33

44
table.onmouseover = function(event) {
55
// перед переходом до нового елемента миша завжди залишає попередній
6-
// якщо вже встановлено currentElem, то ми не залишили попередній <td>,
6+
// якщо вже встановлено currentElem, то ми ще не залишили попередній <td>,
77
// і цей mouseover відбувається всередині, тому ігноруємо подію
88
if (currentElem) return;
99

@@ -16,7 +16,7 @@ table.onmouseover = function(event) {
1616
// ігнорувати
1717
if (!table.contains(target)) return;
1818

19-
// ура! ми перейщли до нового <td>
19+
// ура! ми перейшли до нового <td>
2020
currentElem = target;
2121
onEnter(currentElem);
2222
};

0 commit comments

Comments
 (0)