2014-04-27 3 views
0

Je suis nouveau et j'essaie de penser plus fonctionnellement et de nettoyer mon code-le rendre plus sec ... J'ai essayé d'utiliser ceci avec hover et ça ne marchera pas. Je sais que je manque quelque chose de simple et de facile, mais ... merci d'avance pour de l'aide.Essayer de nettoyer jquery/javascript et penser plus fonctionnellement

$('#phone').on({ 
     mouseenter: function() {$(this).attr('src','../web-marketing/images/PhoneBlue.png')}, 
     mouseleave: function() {$(this).attr('src','../web-marketing/images/Phone4.png')}}); 

    $('#email').on({ 
     mouseenter: function() {$(this).attr('src','../web-marketing/images/EmailBlue.png')}, 
     mouseleave: function() {$(this).attr('src','../web-marketing/images/Email4.png')}}); 

    $('#live-chat').on({ 
     mouseenter: function() {$(this).attr('src','../web-marketing/images/ChatBoxBlue.png')}, 
     mouseleave: function() {$(this).attr('src','../web-marketing/images/ChatBox4.png')}}); 
+0

Avez-vous essayé 'mouseover' au lieu de' mouseenter'? Et 'mouseout' au lieu de' mouseleave'? –

+1

http://codereview.stackexchange.com/ – isherwood

+0

Si le vol stationnaire ne fonctionnait pas, c'était probablement parce que votre syntaxe était erronée. Quelle est la question, exactement? – isherwood

Répondre

0

S'il vous plaît essayer, mais rappelez-vous téléphone, email et chat en direct doit être id de la balise d'image

$('#phone').hover(
    function() {$(this).attr('src','../web-marketing/images/EmailBlue.png');}, 
    function() {$(this).attr('src','../web-marketing/images/Phone4.png');} 
); 


$('#email').hover(
    function() {$(this).attr('src','../web-marketing/images/EmailBlue.png');}, 
    function() {$(this).attr('src','../web-marketing/images/Email4.png');} 
); 


$('#live-chat').hover(
    function() {$(this).attr('src','../web-marketing/images/ChatBoxBlue.png');}, 
    function() {$(this).attr('src','../web-marketing/images/ChatBox4.png');} 
); 
Questions connexes