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

initialism class should use text-capitalize() #15825

Closed
gideoncresswell opened this issue Feb 17, 2015 · 2 comments · Fixed by #15826
Closed

initialism class should use text-capitalize() #15825

gideoncresswell opened this issue Feb 17, 2015 · 2 comments · Fixed by #15826
Labels
Milestone

Comments

@gideoncresswell
Copy link

Currently .initalism in type.less looks like this -

.initialism {
  font-size: 90%;
  text-transform: uppercase;
}

but it should take advantage of .text-capitalize defined above so it should be

.initialism {
  font-size: 90%;
  .text-capitalize();
}

in my opinion 😄

@kkirsche
Copy link
Contributor

I think you mean .text-uppercase() in the second example not Capitalize 👍

kkirsche added a commit to kkirsche/bootstrap that referenced this issue Feb 17, 2015
[Fixes twbs#15825] .initialism class should use text-uppercase() LESS mixin.
@gideoncresswell
Copy link
Author

My bad. Thanks.

@mdo mdo added this to the v3.3.4 milestone Feb 27, 2015
mdo added a commit that referenced this issue Feb 27, 2015
[Fixes #15825] .initialism class should use mixin
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 a pull request may close this issue.

3 participants