Why is this bad?…

Say you did the following:

var test = urlParse(“http://example.com”);
var test2 = urlParse(“http://example2.com”);

alert(test.getHost()); // “example2.com”
alert(test2.getHost()); // “example2.com”

console.log(window === test) // true
console.log(test === test2) // true

You either need to create a closure and freeze dry the url in a “hoisted” set of local variables or construct an object with the new keyword.

Thanks for the feedback. The easiest way to get the script above to use an image for the eyes is probably to switch out the this.create function with something along the lines of this:

this.create = function create(x,y,size,append){
		//Create DOM node (an img this time)
		var eye = document.createElement('img');

		eye.src = "/somefolder/myEyeImage.png"; //Image URL here

		eye.style.width=size;
		eye.style.height=size;
		eye.style.position = 'relative';
		eye.style.top = y+'px';
		eye.style.left = x+'px';
		document.getElementById(append).appendChild(eye);
		return eye;
	}

Basically rather than creating a Canvas element and drawing the eye on to it, it creates an image element then sets its src to the location of whatever image you want to be the eye itself. (Just make sure the eye image by default is looking upwards)

Since its a bit of a lazy fix you’ll still have to set the height/width of the image via third parameter of the “new alienEye()” constructor.

Hope that helps,

Carl

Essentially I’ve just changed the container for the dots from an A to a SPAN, then updated the CSS accordingly. Checkout the JavaScript and CSS to see the fixed version of the code .