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

Breadcrumbs broken in IE7 #2224

Closed
drakej opened this issue Feb 23, 2012 · 6 comments
Closed

Breadcrumbs broken in IE7 #2224

drakej opened this issue Feb 23, 2012 · 6 comments
Milestone

Comments

@drakej
Copy link

drakej commented Feb 23, 2012

http://twitter.github.com/bootstrap/components.html#breadcrumbs

Open the following URL in IE7 (I used IE9 with IE7 rendering mode) and notice how the list style is set to disc on the

  • tags and the display property is set to inline-block.

  • @hihain
    Copy link

    hihain commented Feb 26, 2012

    Can be fixed by using inline instead of inline-block. This will even let the list style disappear.

    Replace
    .breadcrumb li {
    display: inline-block;

    with
    .breadcrumb li {
    display: inline;

    in the breadcrump component. Tested with IE 7-9, FF10, Opera 11.61, Safari 5.1.2, Chrome...

    @mdo
    Copy link
    Member

    mdo commented Feb 28, 2012

    Resolved in 2.0.2-wip.

    @jij
    Copy link

    jij commented Feb 28, 2012

    it doesn't solve IE issue completely. As display: inline-block; is not working in IE7. IE7 hack needed:

    .breadcrumb li {
    zoom:1; *display: inline;

    @jij
    Copy link

    jij commented Feb 28, 2012

    .breadcrumb li {
    display: inline;

    works better if it doesn't break anything else

    @djupudga
    Copy link

    djupudga commented Mar 1, 2012

    .breadcrumb li {
    display: inline-block;
    zoom:1; *display: inline;

    Seems to work in ie7,8,9, firefox, safari, chrome and opera.

    @mdo
    Copy link
    Member

    mdo commented Mar 5, 2012

    Added the .ie7-inline-block mixin to resolve. Thanks!

    @mdo mdo closed this as completed Mar 5, 2012
    mdo added a commit that referenced this issue Mar 5, 2012
    stempler pushed a commit to stempler/bootstrap that referenced this issue Apr 11, 2014
    stempler pushed a commit to stempler/bootstrap that referenced this issue Apr 11, 2014
    stempler pushed a commit to stempler/bootstrap that referenced this issue Nov 4, 2014
    stempler pushed a commit to stempler/bootstrap that referenced this issue Nov 4, 2014
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    None yet
    Projects
    None yet
    Development

    No branches or pull requests

    5 participants