Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

JavaScript jQuery Basics (2014) Creating a Simple Lightbox Perform: Part 4

daniel halen
daniel halen
2,540 Points
Posted by daniel halen
daniel halen
2,540 Points

the function dont work. no lightbox added

i followed step by step, but my code in workspace dont work like the video does. i would need some help to see what did go wrong in my code. are you able to see my workspace code or i need to send it?

<!DOCTYPE html>
<html>
<head>
    <link rel="stylesheet" href="css/style.css" type="text/css" media="screen" title="no title" charset="utf-8">
    <title>Image Gallery</title>
</head>
<body>
    <h1>Image Gallery</h1>
    <ul id="imageGallery">
        <li><a href="images/refferal_machine.png"><img src="images/refferal_machine.png" width="100" alt="Refferal Machine By Matthew Spiel"></a></li>
        <li><a href="images/space-juice.png"><img src="images/space-juice.png" width="100" alt="Space Juice by Mat Helme"></a></li>
        <li><a href="images/education.png"><img src="images/education.png" width="100" alt="Education by Chris Michel"></a></li>
        <li><a href="images/copy_mcrepeatsalot.png"><img src="images/copy_mcrepeatsalot.png" width="100" alt="Wanted: Copy McRepeatsalot by Chris Michel"></a></li>
        <li><a href="images/sebastian.png"><img src="images/sebastian.png" width="100" alt="Sebastian by Mat Helme"></a></li>
        <li><a href="images/skill-polish.png"><img src="images/skill-polish.png" width="100" alt="Skill Polish by Chris Michel"></a></li>
        <li><a href="images/chuck.png"><img src="images/chuck.png" width="100" alt="Chuck by Mat Helme"></a></li>
        <li><a href="images/library.png"><img src="images/library.png" width="100" alt="Library by Tyson Rosage"></a></li>
        <li><a href="images/boat.png"><img src="images/boat.png" width="100" alt="Boat by Griffin Moore"></a></li>
        <li><a href="images/illustrator_foundations.png"><img src="images/illustrator_foundations.png" width="100" alt="Illustrator Foundations by Matthew Spiel"></a></li>
        <li><a href="images/treehouse_shop.jpg"><img src="images/treehouse_shop.jpg" width="100" alt="Treehouse Shop by Eric Smith"></a></li>
    </ul>
    <script src="http://code.jquery.com/jquery-1.11.0.min.js" type="text/javascript" charset="utf-8"></script>
    <script src="js/app.js" type="text/javascript" charset="utf-8"></script>
    <div id="overlay"></div>
</body>
</html>

app.js

//problem: User when clicking to image
//soulution: create an overlay

var $overlay = $('<div id="overlay"></div>');
var $image = $("<img>");
var $caption = $("<p></p>");

//an image to overlay
$overlay.append($image);

//a caption
$overlay.append($caption);
//Add overlay
$("body").append($overlay);


//capture the click event
$("imageGallery a").click(function() {
  event.preventDefault();
  var imageLocation = $(this).attr("href");
 //update overlay with image linked
  $image.attr("src", imageLocation);

  //show the overlay
  $overlay.show();



  //get child's alt attribute
var captionText = $(this).children("img").attr("alt");
$caption.text(captionText);                           
});                         

 //when overlay is clicked
$overlay.click(function() {
  //Hide the overlay
  $(this).hide();
});

style.css

body {
    font-family: sans-serif;
    background: #384047;
}
h1 {
    color: #fff;
    text-align: center
}

ul {
    list-style:none;
    margin: 0 auto;
    padding: 0;
    display: block;
    max-width: 780px;
    text-align: center;
}
ul li {
    display: inline-block;
    padding: 8px;
    background:white;
    margin:10px;
}
ul li img {
    display: block;
}
a {
    text-decoration: none;
}
/** Start Coding Here **/
#overlay {
 background:rgba(0,0,0,0,7);
  width:100%;
  height:100%;
  position:absolute;
  top:0;
  left:0;
  display:none;
  text-align::center;
}

#overlay img {
 margin-top:10%;

}

#overlay p {
  color:white;
}

Thanks for all help!

Edited for readability and added this code to the question - Dane E. Parchment Jr. (Moderator)

KOJI NAKAGAWA
KOJI NAKAGAWA
11,261 Points
KOJI NAKAGAWA
KOJI NAKAGAWA
11,261 Points

Hi Daniel,

On your app.js, you are missing "#" so JS does not recognize your "imageGallery" as id.

Your code:

$("imageGallery a").click(function() {  
      /*remain code*/
  }

This should work:

$("#imageGallery a").click(function() { 
      /*remain code*/
  }

I hope it helps!

Koji

1 Answer

Karl Jones
Karl Jones
8,761 Points
Karl Jones
Karl Jones
8,761 Points

Another thing: $("#imageGallery a").click(function(YOU NEED TO ADD 'event' HERE) { event.preventDefault();