danj707/fantasy-app

simplify these event handlers

Opened this issue · 0 comments

https://github.com/danj707/fantasy-app/blob/master/public/app.js#L808

All these are identical, aside from the position. You can add a data attribute, or use an if/else to set the position, and then do the rest of the things. That'll cut down the number of event handlers you have to bind