Упростите избыточный код jQuery - PullRequest
Новая
       6

Упростите избыточный код jQuery

4 голосов
/ 10 февраля 2011

Я хотел бы упростить мой код jQuery для анимации всплывающего окна / всплывающей подсказки нижнего колонтитула, потому что он избыточен и не очень расширяем. Я нашел этот пост: код jQuery для упрощения , но не смог понять, как применить его в моей ситуации. Спасибо! 

<div id="footer-wrap">
    <div id="footer">
        <ul>
            <li class="copyright"><a href="#" >copyright</a></li>
            <li class="facebook"><a href="#" target="_blank">facebook</a></li>
            <li class="twitter"><a href="#" target="_blank">twitter</a></li>
            <li class="wordpress"><a href="#" target="_blank">blog</a></li>
        </ul>       
    </div>
    <div class="popup">
        <p class="popup-wordpress"><span class="popup-icon"></span><span class="popup-text">Check Out Our Blog</span></p>
        <p class="popup-twitter"><span class="popup-icon"></span><span class="popup-text">Follow us on Twitter</span></p>
        <p class="popup-facebook"><span class="popup-icon"></span><span class="popup-text">Become a fan on Facebook</span></p>
        <p class="popup-copyright"><span class="popup-text">Copyright &copy; 2011<br />All Rights Reserved</span></p>
    </div>
</div>


$(function() {
   $('.copyright').hover(
    function() {
        $('.popup-copyright').stop().animate({ marginTop: -52 }, 100);
    },
    function() {
        $('.popup-copyright').stop().animate({ marginTop: 0 }, 100);
});
$('.copyright').click(function() {
        return false
    });
$('.facebook').hover(
    function() {
        $('.popup-facebook').stop().animate({ marginTop: -52 }, 100);
    },
    function() {
        $('.popup-facebook').stop().animate({ marginTop: 0 }, 100);
});
$('.twitter').hover(
    function() {
        $('.popup-twitter').stop().animate({ marginTop: -52 }, 100);
    },
    function() {
        $('.popup-twitter').stop().animate({ marginTop: 0 }, 100);
});
$('.wordpress').hover(
    function() {
        $('.popup-wordpress').stop().animate({ marginTop: -52 }, 100);
    },
    function() {
        $('.popup-wordpress').stop().animate({ marginTop: 0 }, 100);
});
    });

1 Ответ

9 голосов
/ 10 февраля 2011

Если это будут единственные классы на элементах, которые вы обнаружите, вы можете сделать это: 

$('.copyright,.facebook,.twitter,.wordpress').hover(
    function() {
        $('.popup-' + this.className).stop().animate({ marginTop: -52 }, 100);
    },
    function() {
        $('.popup-' + this.className).stop().animate({ marginTop: 0 }, 100);
});
$('.copyright').bind('click',false);

или даже немного короче, как это: 

$('.copyright,.facebook,.twitter,.wordpress').hover( function(e) {
    $('.popup-' + this.className).stop().animate({ marginTop: e.type === 'mouseenter' ? -52 : 0 }, 100);
});
$('.copyright').bind('click',false);

Обратите внимание, что .bind('click',false); требует jQuery 1.4.3 или более поздней версии.

... или еще лучше, используйте метод delegate() (документы) . 

$('#footer').delegate('li','hover' function(e) {
    $('.popup-' + this.className).stop().animate({ marginTop: e.type === 'mouseenter' ? -52 : 0 }, 100);
});
$('.copyright').bind('click',false);
...