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

btn-large height changes with icon in btn-group #730

Closed
misino opened this issue Feb 2, 2013 · 5 comments
Closed

btn-large height changes with icon in btn-group #730

misino opened this issue Feb 2, 2013 · 5 comments
Assignees
Milestone

Comments

@misino
Copy link

misino commented Feb 2, 2013

When you have 2 large buttons in btn-group and in first button is icon and in second buttons there isnt icon. The button with icon is a little bit larger.

My configurations Ubuntu, Firefox (version 18).

You can see difference on following image.

font-awesome

@tagliala
Copy link
Member

tagliala commented Feb 2, 2013

Same here (bootstrap 2.2.2 + fontawesome 3.0.2 + ubuntu 12.10 + firefox 18): http://jsfiddle.net/5EHaL/

This is an issue related to: #635 and the effin line-height: normal;

Since .btn is fine and the issue seems related to .btn-large, a simple workaround is:

.btn-large i[class^=icon] {
    line-height: inherit !important;
}

http://jsfiddle.net/5EHaL/3/

Note: this workaround may have unwanted side effects

PS: why are you using span instead of i for icons?

@davegandy
Copy link
Member

Hmm. I'll look into this for 3.1.0.

@thomaskraus
Copy link

This workaround will preserve the original line-height:

.btn-large i[class^=icon] {
  float: left;
  line-height: inherit !important;
}

@ghost ghost assigned tagliala Aug 21, 2013
@tagliala
Copy link
Member

I think this one will be fixed in 4.0.0 with the new syntax. It should be due to *[class="icon-"] stuff

@davegandy
Copy link
Member

Confirmed, fixed in 4.0.0-wip branch. Thanks for adding the visual test, @tagliala!

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

4 participants