Как стать автором
Обновить

Комментарии 12

Перед animate обычно используют stop, иначе получится как у вас сейчас — поводите быстро мышкой и уведите ее от меню… а оно живет своей жизнью
Благодарю, исправил.
То, что данный скрипт будет корректно работать с многострочным меню – и не было заявлено.
НЛО прилетело и опубликовало эту надпись здесь
Зависит от размера окна браузера (с) Кэп
Мне кажется пора уже потихоньку переходить на css transitions — точно такой же эффект можно уже сделать без единой строчки javascript. Представьте насколько легче будет ваша страничка, а волосы гладкими и шелковистыми!
Добавил вашему примеру более приятную анимацию с помошью «jquery easing»
jsfiddle.net/bYY6Y/57/
А по поводу конструктивной критики:
1. Анимация у вас при разных событиях ведь одинаковая, поетому следует написать одну лишь функцию и вызывать ее когда это нужно.
2. Вы постоянно обращаетесь к одним и тем же элементам и у вас постоянно идет поиск по DOM — это неправильно. Следует закешировать эти элементы в переменных
3. Ну и почитайте про событие .on()
jsfiddle.net/bYY6Y/71/
Благодарю, полезные сведения.
Добавил информацию в пост.
Только полноправные пользователи могут оставлять комментарии. Войдите, пожалуйста.