In the end this comes down to personal preference and ease of use, we've been doing this for a while now without any push back on PR's. Using underscores for IDs and dashes for classes makes things unnecessary complex and slows down everyone who wants to take these 'rules' into account. There are many different answers to this question but I do not see the advantage of using that style.