Ah, yeah, definitely should be a data attribute. I knew something was off when I wrote it but I couldn't put my finger on it :)