Jquery not executing

the code seems to be working when i do;
$(document.body).click(function ()

but not with a specific id


<!DOCTYPE>
<html>
	<head>
		<title>TEST</title>
		<link href="style.css" media="screen" rel="stylesheet" type="text/css" />
                     <script src="http://code.jquery.com/jquery-latest.min.js"></script>
	</head>
	<body>
	<script>

	$('#location').click(function ()
	{
	if ($("#aa:first").is(":hidden"))
	{
	$("#aa").slideDown("slow");
	}
	else
	{$("#aa").hide();
	}
});</script>




 <div id="location" class="location"><a href="#"> <div id="aa"></div></div>



	</body>

</html>

Open in new window

fb19071907Asked:
Who is Participating?

[Webinar] Streamline your web hosting managementRegister Today

x
 
leakim971Connect With a Mentor PluritechnicianCommented:
Check this :


<!DOCTYPE>
<html>
<head>
<title>TEST</title>
<link href="style.css" media="screen" rel="stylesheet" type="text/css" />
<script src="http://code.jquery.com/jquery-latest.min.js"></script>
<script>
	$(document).ready(function() {
		$('#location').click(function () {
			if ($("#aa:first").is(":hidden")) {
				$("#aa").slideDown("slow");
			}
			else {
				$("#aa").hide();
			}
		});
	});
</script>
</head>
<body>
<div id="location" class="location"><a href="#">Click</a><div id="aa">HELLO</div></div>
</body>
</html>

Open in new window

0
 
CyberGhostCommented:
I believe you're missing a closing </a> tag in your code. Other than that, I would relocate your script into <head> part of code, just as asleakim971 pointed out.
0
 
SRigneyCommented:
inline javascript like that will work, but it must be after the item that it's trying to reference.

If you put it into the document ready function that leakim showed you, then the html will be finished loading before the javascript runs making the order not important.


CyberGhost, the recommendations for speed is CSS at the top of the page and javascript at the bottom of the page.
0
 
leakim971PluritechnicianCommented:
Thanks for the points!
0
All Courses

From novice to tech pro — start learning today.