Comments 12
Перед animate обычно используют stop, иначе получится как у вас сейчас — поводите быстро мышкой и уведите ее от меню… а оно живет своей жизнью
Судя по анимации, вместо выделения ромбиком, вам нужно было сделать Pacman'а:
ПУНКТ2 ПУНКТ3 ПУНКТ4
ПУНКТ2 ПУНКТ3 ПУНКТ4Вот так лучше: jsfiddle.net/bYY6Y/19/
Мне кажется пора уже потихоньку переходить на css transitions — точно такой же эффект можно уже сделать без единой строчки javascript. Представьте насколько легче будет ваша страничка, а волосы гладкими и шелковистыми!
Добавил вашему примеру более приятную анимацию с помошью «jquery easing»
jsfiddle.net/bYY6Y/57/
jsfiddle.net/bYY6Y/57/
А по поводу конструктивной критики:
1. Анимация у вас при разных событиях ведь одинаковая, поетому следует написать одну лишь функцию и вызывать ее когда это нужно.
2. Вы постоянно обращаетесь к одним и тем же элементам и у вас постоянно идет поиск по DOM — это неправильно. Следует закешировать эти элементы в переменных
3. Ну и почитайте про событие .on()
jsfiddle.net/bYY6Y/71/
1. Анимация у вас при разных событиях ведь одинаковая, поетому следует написать одну лишь функцию и вызывать ее когда это нужно.
2. Вы постоянно обращаетесь к одним и тем же элементам и у вас постоянно идет поиск по DOM — это неправильно. Следует закешировать эти элементы в переменных
3. Ну и почитайте про событие .on()
jsfiddle.net/bYY6Y/71/
Sign up to leave a comment.
«Плавающее» меню на jQuery для начинающих