The reason for using `.attr('data-id'` is if you're going to read it OUTSIDE of jquery. Which we're not. The code on line 294 is also jquery. 
I'd be happier if it was `.data('id', ...)`