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

Support .h1 in place of h1 inside .jumbotron #11299

Merged
merged 1 commit into from
Nov 30, 2013
Merged

Support .h1 in place of h1 inside .jumbotron #11299

merged 1 commit into from
Nov 30, 2013

Conversation

kevinoid
Copy link
Contributor

As in #7284, it can often be useful to style text like a heading in
places where using the heading tag would be either semantically
incorrect, less accessible, or otherwise invalid.

Support this in .jumbotron as in the rest of the document by allowing
elements with class .h1 to stand in for the h1 element.

@zlatanvasovic
Copy link
Contributor

You don't follow styleguides. You need to divide selectors by lines. So, not h1, .h1, it should be:

h1,
.h1

@kevinoid
Copy link
Contributor Author

You are right. I followed the existing style where .h1 is used with h1 (in less/type.less:78 and less/type.less:116). If you would prefer that I change it, I can resubmit the pull request.

@zlatanvasovic
Copy link
Contributor

Don't resubmit request, just rebase commit.

2013/10/31 Kevin Locke notifications@github.com

You are right. I followed the existing style where .h1 is used with h1 (in
less/type.less:78 and less/type.less:116). If you would prefer that I
change it, I can resubmit the pull request.


Reply to this email directly or view it on GitHubhttps://github.com//pull/11299#issuecomment-27487399
.

Zlatan Vasović - ZDroid

As in #7284, it can often be useful to style text like a heading in
places where using the heading tag would be either semantically
incorrect, less accessible, or otherwise invalid.

Support this in .jumbotron as in the rest of the document by allowing
elements with class .h1 to stand in for the h1 element.

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
@kevinoid
Copy link
Contributor Author

Sure thing. How does this look?

@zlatanvasovic
Copy link
Contributor

Looks good. :)

@mdo mdo mentioned this pull request Nov 30, 2013
@mdo mdo merged commit 441f9d6 into twbs:master Nov 30, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants