alishalisha / canvas-stuff

Messing around with just canvas.
1 stars 0 forks source link

refactor: move event handlers to activateTool function #3

Closed alishalisha closed 9 years ago

alishalisha commented 9 years ago

activateTool function should accept a data attribute selector as a parameter, i.e.

var el = $('#canvas-controls');

el.click(function(event) {
   activateTool(Object.keys(event.target.dataset));
});

activateTool = function(tool) {
   var target = $(event.target);
   var activeClass = 'active-'+tool;
   if (target.is('a')) {
     target.addClass(activeClass).siblings().removeClass(activeClass);
   } else {
       return
   }
}

// this is gibberish for now, alisha is sleepy and will continue tomorrow :)

:sleeping: :sleepy: :zzz: :pray:

banderson623 commented 9 years ago

:+1: