Skip to content
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

[Header] Sub Header inside Header Should Force Display: Block #3020

Closed
scoman81 opened this issue Sep 13, 2015 · 3 comments
Closed

[Header] Sub Header inside Header Should Force Display: Block #3020

scoman81 opened this issue Sep 13, 2015 · 3 comments
Milestone

Comments

@scoman81
Copy link

The icon header examples have invalid HTML markup, as div is not allowed inside an H2 tag, and won't validate.

@jlukic
Copy link
Member

jlukic commented Sep 13, 2015

Don't worry validation is meaningless. 😄

In all seriousness you can just use a span if it bothers you

@scoman81
Copy link
Author

I kind of agree about the validation (personally) but I'm sure some people will have project requirements they have to meet for clients.

Using a span requires some extra markup, because it doesn't do line breaks like a div does. It's easy to do, but I thought I'd mention it due to being another exception to the tag-agnostic approach of SUI. I'm guessing there's a CSS way to make a span work the same as a div in this case.

@jlukic
Copy link
Member

jlukic commented Sep 16, 2015

I think it makes sense to force display: block on sub header to allow it to work correctly regardless of tag.

@jlukic jlukic changed the title Icon headers (and more) invalid HTML markup [Header] Sub Header inside Header Should Force Display: Block Sep 16, 2015
@jlukic jlukic added this to the 2.1.5 milestone Sep 16, 2015
@jlukic jlukic modified the milestones: 2.1.5, 2.1.7 Nov 6, 2015
@jlukic jlukic closed this as completed Dec 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants