Jonathan - Fixes

[soggybag]

shared-canvas/jonathan.js

Line 35 in e393e3c

function drawRect(x, y, width, height, color) {

Great work. I like your functions. These are a really great idea. You could take this up a notch by including the ctx as a parameter. Without this each of these function assumes there is a variable ctx in scope which may not be guaranteed if you used them in another project.

Fix this by injecting the dependency, which is a fancy way of saying make ctx a parameter.

function drawRect(ctx, x, y, width, height, color) {
  ctx.beginPath()
  ...
}

Here you can get your ctx anywhere and it could have any name:

drawRect(document.querySelector('canvas'), 100, 20, 23, 44, 'red')

[jewarner57]

Resolved.