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

IE nav-justified issue. #10649

Closed
voviklenin opened this issue Sep 15, 2013 · 17 comments
Closed

IE nav-justified issue. #10649

voviklenin opened this issue Sep 15, 2013 · 17 comments
Labels

Comments

@voviklenin
Copy link

There is a problem with nav-justified when opening it in IE, the dropdown item falls 1 line lower than the rest of menu items. Works fine on mozilla and chrome.

@cvrebert
Copy link
Collaborator

What version(s) of IE?

@poromaa
Copy link

poromaa commented Sep 16, 2013

~~Actually I get same problem if i just resize the browser-window in chrome (29.0.1547.65) + safari (Version 6.0.5 (8536.30.1)). Even on getbootstrap.com.
To replicate problem:

  1. goto http://getbootstrap.com/components/#nav-justified
  2. resize browser window width to smallest (400px in Chrome)
  3. resize it back to normal width and now all "home" tabs are above the other two tabs in the example~~

@cvrebert
Copy link
Collaborator

@poromaa That's #9774. @voviklenin's issue sounds different.

@voviklenin: Also, could you post a JSFiddle or JSBin live example of the problem?

@voviklenin
Copy link
Author

The Version of Explorer is 10, I will try to post it on JSFiddle but within next week or so, little bit busy with the project.

@voviklenin
Copy link
Author

Forgot to mension that I am using wordpress with wp-bootstrap-navwalker.

@voviklenin
Copy link
Author

It also creates a glitch in mobile button slider, the submenu does drop down but not in smaller window on the left like in standard menu.

@cvrebert
Copy link
Collaborator

Well, we can't help with the WordPress-specific parts, but if you can create the plain JSFiddle example, then we can certainly debug that.

@voviklenin
Copy link
Author

@voviklenin
Copy link
Author

Here is an example.

@voviklenin
Copy link
Author

            <div class="row">
                <div class="col-md-12 rempd">
                    <nav class="navbar navbar-default navbar-turquoise" role="navigation">
                        <div class="navbar-header">
                                <button type="button" class="navbar-toggle" data-toggle="collapse" data-target=".navbar-ex1-collapse">
                                    <span class="sr-only">Toggle navigation</span>
                                    <span class="icon-bar"></span>
                                    <span class="icon-bar"></span>
                                    <span class="icon-bar"></span>
                                </button>
                        </div>

                        <div class="collapse navbar-collapse navbar-ex1-collapse">
                                <?php
                                    wp_nav_menu( array(
                                        'menu'              => 'nav_main',
                                        'theme_location'    => 'nav_main',
                                        'depth'             => 2,
                                        'container'         => false,
                                        'menu_class'        => 'nav nav-justified',
                                        'fallback_cb'       => 'wp_page_menu',
                                        'walker'            => new wp_bootstrap_navwalker())
                                        );        
                                ?>
                        </div><!-- /.navbar-collapse -->
                    </nav>
                </div>
            </div>

@voviklenin
Copy link
Author

This is the way it was implemented. I am not sure wether the plugin bootstrap_navwalker was created by same people as the bootstrap itself.

@cvrebert
Copy link
Collaborator

@voviklenin As a member of the Bootstrap core team, I can tell you that it wasn't. We don't write any plugins for third-party stuff like WordPress.

@voviklenin
Copy link
Author

http://jsfiddle.net/voviklenin/hKq5t/

It does cause trouble even if I hardwire and create the justified menu as per tutorial, has the same issue as through the plugin

@lifofernandez
Copy link

Hi, first, sry for my english.

I have the exact same issue here...
I think the problem is not with bootstrap nor the menu waker (http://illuminatikarate.com/blog/bootstrap-walker-for-wordpress/)...
before the menu there are diferences in the way wp-bootstrap code the header...
check this... http://www.diffchecker.com/a1ocvp87
in that way, wp-bootstrap is not ready for nav-justified

im gona give a try on this...

@cvrebert
Copy link
Collaborator

cvrebert commented Oct 8, 2013

Judging by our Justified Nav example page, I don't think we support using .justified-nav on the navbar.

@lifofernandez
Copy link

if you use the code inspector to replace .navbar-nav with .justified-nav in the de ul menu to the bootstrap page it seems to work fine, even in iE8 (but I dont know alrredy if its collapsing right)

@mdo
Copy link
Member

mdo commented Oct 26, 2013

We don't support the .justified-nav class within navbars at this time.

@mdo mdo closed this as completed Oct 26, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants