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

Foundation Top Bar - FA Icon Doesn't Display Correctly #2050

Closed
LDigital84 opened this issue Sep 29, 2013 · 4 comments
Closed

Foundation Top Bar - FA Icon Doesn't Display Correctly #2050

LDigital84 opened this issue Sep 29, 2013 · 4 comments
Milestone

Comments

@LDigital84
Copy link

I know Foundation support is planned for 4.0 (#1212). This is a bug I found with using Font Awesome in Foundation.

I have an icon in my top bar navigation. This icon isn't aligned like my text. Screenshot:

fa-icon

After digging through the CSS I found the problem. In Font Awesome we have the following CSS:

[class^="icon-"], [class*=" icon-"]{
    line-height: normal;
}

This is overriding the CSS for Foundation's Top Bar which is:

.top-bar-section li a:not(.button){
    line-height: 45px;
}

Thus the Font Awesome Icon doesn't display correctly. I was able to correct this using the following CSS:

.top-bar-section li a:not(.button) .icon-search  {
   line-height: 45px !important;
}

I would love to see this issue fixed in 4.0.

Thank You!

@tagliala
Copy link
Member

It should be automatically fixed with the new syntax, because there should be no need of line-height hacks here and there

@tagliala
Copy link
Member

refers to #730

@LDigital84
Copy link
Author

@tagliala - You were correct. I just updated the site I was having this issue on to FA 4.0.1 and it works fine now. =>

@tagliala
Copy link
Member

tagliala commented Nov 2, 2013

@Lynda333 glad to hear that, because unfortunately we had to include line-height: 1 in the base class

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

2 participants