views:

1032

answers:

7

I'm trying to make a div fadeIn when another div is clicked and fadeOut again when another div is clicked (which would be the close button) but my code doesn't work, did I forget something?

Here's the CSS:

body{
    margin: 0;
    padding: 0;
    text-align: center;
    background-color:#f0f2df;
}

#container{
    border: solid 1px #f0f2df;
    background-color:#f0f2df;
    text-align: left;
    margin: auto;
    width: 939px;
    height: 570px;
    top:41px;
    position:relative;
}
#contact_form{
    display: none;
    background-image:url(../images/bg.png);
    width: 703px;
    height: 379px;
    position:absolute;
    left:236px;
    bottom:34px;

}
.contact_close{
    display:none;
    background-image:url(../images/close.png);
    width:17px;
    height:17px;
    position:absolute;
    right:5px;
    top:135px;
}

The HTML:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"&gt;
<html xmlns="http://www.w3.org/1999/xhtml"&gt;
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<link rel="stylesheet" type="text/css" href="css/main.css" />
<title>test</title>
<script type='text/javascript' src='js/jquery.js'></script>
<script type='text/javascript' src='js/click.js'></script>
</head>

<body>
    <div id="container">
        <div class="button_contact"></div>
        <div id="contact_form">
        <div class="button_close"></div></div>
</div>
</body>
</html>

and the JavaScript

$(document).ready(function(){ 

 $("button_contact").click(function() { 
    $("#contact_form").fadeIn("slow");
});

 $(".contact_close").click(function() { 
      $("#contact_form").fadeOut("slow"); 
    });
 });
+2  A: 

you forgot a . before button close...

$(".button_contact")....

should work

akearney
+2  A: 

you need the "." before button_contact

$(document).ready(function(){ 
  $(".button_contact").click(function() { 
    $("#contact_form").fadeIn("slow");
  });

  $(".contact_close").click(function() { 
    $("#contact_form").fadeOut("slow"); 
  });
});
scunliffe
aw man that's embarassing I even double checked it!oh well thank you kind sir.
Bruno
+1  A: 

The selector on your fadeIn button is a bit off. Your original code matches an element with a node name of button_contact not with a class of button_contact.

Try:

$(".button_contact").click(function() { 
    $("#contact_form").fadeIn("slow");
});
Jon Cram
+1  A: 

$("button_contact").click(function() { should be

$(".button_contact").click(function() {
rosscj2533
+1  A: 

Try this:

$(document).ready(function(){ 

 $(".button_contact").click(function() { 
     $("#contact_form").fadeIn("slow");
 });

 $(".button_close").click(function() { 
  $("#contact_form").fadeOut("slow"); 
  });
 });
Vincent Ramdhanie
A: 

I love that kind of errors where you're missing a dot, or something like that, and spend hours fixing the problem :P

Hybryd
+1  A: 

jquery also has a .toggle() function that allows you to pass multi-functions that are "toggled" between each other when the element/s are clicked.

http://api.jquery.com/toggle/ a nice function because you can add as many functions as you want.

$(document).ready(function(){ 
  $(".button_contact").toggle(function() { 
                      $("#contact_form").fadeIn("slow");
                      },
                       function() { 
                     $("#contact_form").fadeOut("slow"); 
                    });
   });
Mike