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.

Sign up for GitHub

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

Jump to bottom

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
css
Milestone
v3.3.4

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
@kkirsche
[Fixes twbs#15825] .initialism class should use mixin
02195d3 
[Fixes twbs#15825] .initialism class should use text-uppercase() LESS mixin.
@kkirsche kkirsche mentioned this issue Feb 17, 2015
Merged
@gideoncresswell
Copy link
Author

My bad. Thanks.

@mdo mdo added this to the v3.3.4 milestone Feb 27, 2015
@cvrebert cvrebert mentioned this issue Feb 27, 2015
Closed
mdo added a commit that referenced this issue Feb 27, 2015
@mdo
Merge pull request #15826 from kkirsche/patch-19
049e86f 
[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
Assignees
No one assigned
Labels
css
Projects
None yet
Milestone
v3.3.4
Development

Successfully merging a pull request may close this issue.

[Fixes #15825] .initialism class should use mixin kkirsche/bootstrap
3 participants
@mdo @kkirsche @gideoncresswell