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 3

Michael Tucker
Michael Tucker
9,481 Points
Posted by Michael Tucker
Michael Tucker
9,481 Points

Cannot get image gallery clickable

I am trying to copy what Andrew has done, but my results are not replicating what the video is showing. What am I missing? 

//Problem: User when clicking on image goes to a dead end. 
//Poor User Experience
//
//Solution: Create an overlay with the large image.   -- Lightbox
var $overlay = $('<div id="overlay"></div>');
var $image = $('<img>');

$overlay.append($image);

// Add overlay
$("body").append($overlay);
  //  An image
  //  A caption

// Capture the click event on a link to an image.
$("#imageGallery a").click(function(event){

  event.preventDefault();

  var imageLocation = $(this).attr("href");
  //  Update overlay with the image linked in the link
  $image.attr("src", imageLocation);

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

  //  Get Child's alt attribute and set caption
});

// When overlay is clicked
  //  Hide the overlay

Thanks,

9 Answers

George Cristian Manea
George Cristian Manea
30,787 Points

Is your css ok? Like in the video?

    background:rgba(0,0,0,0.5);
    height:100%;
    width:100%;
    position:absolute;
    top:10%;
    text-align:center;
}```
George Cristian Manea
George Cristian Manea
30,787 Points
George Cristian Manea
George Cristian Manea
30,787 Points

First your $image and $overlay don't have values, it should be $image = $('<img>') and $overlay = $('<div id="overlay"></div>'). Second check if some of the code is commented because the way you posted your code it appears so.

Michael Tucker
Michael Tucker
9,481 Points
Michael Tucker
Michael Tucker
9,481 Points

Just reformated my post. I didn't realize about using the backticks. I hope that makes it clearer.

Am I still missing it?

Michael Tucker
Michael Tucker
9,481 Points
Michael Tucker
Michael Tucker
9,481 Points

Just reformated my post. I didn't realize about using the backticks. I hope that makes it clearer.

George Cristian Manea
George Cristian Manea
30,787 Points
George Cristian Manea
George Cristian Manea
30,787 Points

Do you have the jquery file linked to the index.html?

Michael Tucker
Michael Tucker
9,481 Points
Michael Tucker
Michael Tucker
9,481 Points

Yes, it is. It was already linked from the initial workspace.

George Cristian Manea
George Cristian Manea
30,787 Points
George Cristian Manea
George Cristian Manea
30,787 Points

can you post the html code pls?

Michael Tucker
Michael Tucker
9,481 Points
Michael Tucker
Michael Tucker
9,481 Points 
<!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>

</body>
</html>
George Cristian Manea
George Cristian Manea
30,787 Points

You are working in a workspace or local on your PC?

Michael Tucker
Michael Tucker
9,481 Points
Michael Tucker
Michael Tucker
9,481 Points

That was it! I forgot to assign the value none to display! I was sitting here looking through the code know that I had to get rid of the overlay, but trying jQuery was just a dead end loop. Thanks!