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 Drawing Application Perform: Part 5

Colin Sharkey
Colin Sharkey
14,631 Points

jQuery draw canvas not working

https://teamtreehouse.com/workspaces/30285982#

I cant see any errors but the mousedown doesnt seem to be working, not drawing on the canvas.

Any help would be appreciated!

Colin Sharkey
Colin Sharkey
14,631 Points

//Problem: No user interaction causes no change to application //Solution: When user interacts cause changes appropriately

var color = $(".selected").css("background-color"); var $canvas = $("canvas"); var context = $("canvas")[0].getContext("2d"); var lastEvent; var mouseDown = false;

//When clicking on control list items $(".controls").on("click", "li", function(){ //Deselect sibling elements $(this).siblings().removeClass("selected"); //Select clicked element $(this).addClass("selected"); //cache current color color = $(this).css("background-color"); });

//When new color is pressed $("#revealColorSelect").click(function(){ //Show color select or hide the color select changeColor(); $("#colorSelect").toggle(); });

//update the new color span
function changeColor() { var r = $("#red").val(); var g = $("#green").val(); var b = $("#blue").val(); $("#newColor").css("background-color", "rgb(" + r + "," + g +", " + b + ")"); }

//When color sliders change $("input[type=range]").change(changeColor);

//When add color is pressed $("#addNewColor").click(function(){ //Append the color to the controls ul var $newColor = $("<li></li>"); $newColor.css("background-color", $("#newColor").css("background-color")); $(".controls ul").append($newColor); //Select the new color
$newColor.click(); });

//On mouse events on the canvas $canvas.mousedown(function(e){ lastEvent = e; mouseDown = true; }).mousemove(function(e){

//Draw lines if(mouseDown) { context.beginPath(); context.moveTo(lastEvent.offsetX, lastEvent.offsetY); context.moveTo(e.offsetX, e.offsetY); context.strokeStyle = color; context.stroke(); lastEvent = e;
} }).mouseup(function(){ mouseDown = false; }).mouseleave(function(){ $canvas.mouseup(); });

1 Answer

Adrien Contee
Adrien Contee
4,875 Points

All your code is correct. Reformat your code... the comments everywhere are commenting out some of your code. I did a copy and paste and that's all I could find.