-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
hgroup no longer part of html5 spec #1
Comments
Yep, I was familiar with that. Just no time to get back to this project. The thing is... it makes absolutely no difference if people use this in production. You can make up as many fake tags as you want and browsers and screen readers will just treat them like divs, so it's really not that important unless there is a more accessible alternative. |
Fair enough - and impressively rapid response. In fact, I'm trying to wade through best practise to target screen readers and accessibility for blind people, so it's a learning minefield of conflicting info and best practise, but I've been finding your site and StackOverflow comments particularly helpful, so, thanks again! |
No problem. Oh and don't get the wrong idea... I appreciate the feedback, I will definitely make the change so thanks for the reminder. I was just saying that it's not the most urgent thing, that's all. :) |
👍 |
No need to close it, I will fix it, so let's leave this open for now. Thanks. :) |
I noticed hgroup at https://github.com/impressivewebs/Easy-HTML5-Template/blob/master/index.html#L50 but I also noticed http://html5doctor.com/the-hgroup-element/ which says:
The text was updated successfully, but these errors were encountered: