Pull to refresh

Comments 12

Перед animate обычно используют stop, иначе получится как у вас сейчас — поводите быстро мышкой и уведите ее от меню… а оно живет своей жизнью
Благодарю, исправил.
Судя по анимации, вместо выделения ромбиком, вам нужно было сделать Pacman'а:
image ПУНКТ2 ПУНКТ3 ПУНКТ4
То, что данный скрипт будет корректно работать с многострочным меню – и не было заявлено.
Зависит от размера окна браузера (с) Кэп
Мне кажется пора уже потихоньку переходить на css transitions — точно такой же эффект можно уже сделать без единой строчки javascript. Представьте насколько легче будет ваша страничка, а волосы гладкими и шелковистыми!
А по поводу конструктивной критики:
1. Анимация у вас при разных событиях ведь одинаковая, поетому следует написать одну лишь функцию и вызывать ее когда это нужно.
2. Вы постоянно обращаетесь к одним и тем же элементам и у вас постоянно идет поиск по DOM — это неправильно. Следует закешировать эти элементы в переменных
3. Ну и почитайте про событие .on()
jsfiddle.net/bYY6Y/71/
Благодарю, полезные сведения.
Добавил информацию в пост.
Only those users with full accounts are able to leave comments. Log in, please.