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

Drop the $small-font-size variable for the newer $font-size-sm one #20867

Merged
merged 1 commit into from
Oct 9, 2016

Conversation

mdo
Copy link
Member

@mdo mdo commented Oct 9, 2016

Originally $small-font-size was added in v3 to set the font-size of the <small> element. Now that we have newer, global type variables, we can use those instead for these kind of things.

Fixes #20859

Originally -font-size was added in v3 to set the font-size of the <small> element. Now that we have newer, global type variables, we can use those instead for these kind of things.

Fixes #20859
@mdo mdo added this to the v4.0.0-alpha.5 milestone Oct 9, 2016
@mdo mdo merged commit 3b13f81 into v4-dev Oct 9, 2016
@mdo mdo mentioned this pull request Oct 9, 2016
@mdo mdo deleted the dedupe-sm-vars branch October 9, 2016 22:00
@bradly
Copy link
Contributor

bradly commented Oct 10, 2016

@mdo So this changes the way works inside an h* element. Here is the example in the docs: http://v4-alpha.getbootstrap.com/content/typography/#customizing-headings . What do you think about setting `h1, h2, h3, h4, h5, h6 > small { font-size: 80% }? I can create a pull request if you like this approach. Thanks.

@mdo
Copy link
Member Author

mdo commented Oct 10, 2016

Oh snap, I forgot about that part. I'll revisit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants