CODEWITHSUNDEEP
javascripthtmlcanvas
user121168
user121168

Reputation: 41

Draw and fill random colored circles onclick

I am using a canvas element on my web page, and need to be able to draw circles on mouse click with a different color each click. Here is the code I have, which works on some sites but not others. My question is what am I missing for this to work? I know that it only has one color currently in the code, but as it stands, I click all over the canvas and nothing happens.

var canvas = document.getElementById("cirCanvas");
var context = canvas.getContext("2d");

function createImageOnCanvas(imageId) {
  canvas.style.display = "block";
  document.getElementById("circles").style.overflowY = "hidden";
  var img = new Image(300, 300);
  img.src = document.getElementById(imageId).src;
  context.drawImage(img, (0), (0)); //onload....
}

function draw(e) {
  var pos = getMousePos(canvas, e);
  posx = pos.x;
  posy = pos.y;
  context.fillStyle = "#000000";
  context.beginPath();
  context.arc(posx, posy, 50, 0, 2 * Math.PI);
  context.fill();
}

function getMousePos(canvas, evt) {
  var rect = canvas.getBoundingClientRect();
  return {
    x: evt.clientX - rect.left,
    y: evt.clientY - rect.top
  };
}

window.draw = draw;
<head>
  <title></title>
  <meta charset="utf-8">
  <script src="clickCircle.js"></script>

</head>

<body>
  <div id="circles"></div>
  <canvas id="cirCanvas" width="250" height="250" onclick="draw  (event)">
  </canvas>

</body>

Upvotes: 1

Views: 1820

Answers (3)

MaxZoom
MaxZoom

Reputation: 7753

Below is example of a random color generator function.
Also you should try add the click event listener in an unobtrusive way.

var canvas = document.getElementById("cirCanvas");
var context = canvas.getContext("2d");

canvas.addEventListener('click', draw, false);

function draw(e) {
  var pos = getMousePos(canvas, e);
  posx = pos.x;
  posy = pos.y;
  context.fillStyle = randomColor();
  context.beginPath();
  context.arc(posx, posy, 50, 0, 2 * Math.PI);
  context.fill();
}

function randomColor() {
  var color = [];
  for (var i = 0; i < 3; i++) {
    color.push(Math.floor(Math.random() * 256));
  }
  return 'rgb(' + color.join(',') + ')';
}

function getMousePos(canvas, evt) {
  var rect = canvas.getBoundingClientRect();
  return {
    x: evt.clientX - rect.left,
    y: evt.clientY - rect.top
  };
}
#cirCanvas {
  border: 1px solid black;
}
<div id="circles">
  <canvas id="cirCanvas" width="250" height="250"></canvas>
</div>

Upvotes: 3

mplungjan
mplungjan

Reputation: 178413

Here is how to change color - you can randomize the colors:

Best way to generate a random color in javascript?

or use an array of fixed colors

  colors = ["001100","AA0000","00BB00"];

using 

  context.fillStyle = "#"+colors[Math.round(Math.random()*colors.length)];

Random version:

var canvas = document.getElementById("cirCanvas"),
  context = canvas.getContext("2d");


function createImageOnCanvas(imageId) {
  canvas.style.display = "block";
  document.getElementById("circles").style.overflowY = "hidden";
  var img = new Image(300, 300);
  img.src = document.getElementById(imageId).src;
  context.drawImage(img, (0), (0)); //onload....
}

function draw(e) {
  var pos = getMousePos(canvas, e);
  posx = pos.x;
  posy = pos.y;
  context.fillStyle = '#'+(0x1000000+(Math.random())*0xffffff).toString(16).substr(1,6)
  context.beginPath();
  context.arc(posx, posy, 50, 0, 2 * Math.PI);
  context.fill();
}

function getMousePos(canvas, evt) {
  var rect = canvas.getBoundingClientRect();
  return {
    x: evt.clientX - rect.left,
    y: evt.clientY - rect.top
  };
}

//window.draw = draw;
<div id="circles"></div>
<canvas id="cirCanvas" width="250" height="250" onclick="draw(event)">
</canvas>

Upvotes: 1

PRogalla
PRogalla

Reputation: 236

You have to implement a function which is able to generate a random color.

just use sth. like this:

function getRandomColor() {
  var letters = '0123456789ABCDEF';
  var color = '#';
  for (var i = 0; i < 6; i++) {
    color += letters[Math.floor(Math.random() * 16)];
  }
  return color;
}

JsFiddle example

Upvotes: 0

Related Questions