is it ok to rely on a class for data attributes? It seems like using an id would be more stable