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

Problem displaying all navbar links when viewed on phone #10601

Closed
webbyhall opened this issue Sep 12, 2013 · 17 comments
Closed

Problem displaying all navbar links when viewed on phone #10601

webbyhall opened this issue Sep 12, 2013 · 17 comments

Comments

@webbyhall
Copy link

I have a navbar with 8 links. I've noticed that when I activate the navbar on a phone and view it in landscape, it doesn't scroll all the way down to display the last link.

A live example of this can be shown on the 'Jumbotron' example template. When I drop down the menu I can't view the whole login form.

@cvrebert
Copy link
Collaborator

What device and web browser are you using?

@webbyhall
Copy link
Author

I'm using Firefox on HTC V phone.

@cvrebert
Copy link
Collaborator

What version of Firefox? What version of Android?

@webbyhall
Copy link
Author

Sorry, I'm running FF 23.0 and Android 4.0.3.

@cvrebert
Copy link
Collaborator

How far can you scroll? Could you post a screenshot?

@webbyhall
Copy link
Author

I can get half way down the second input field.

image

@mdo
Copy link
Member

mdo commented Sep 13, 2013

Can you scroll to view it?

@ipatovanton
Copy link

same problem on blackberry 9900

@cvrebert
Copy link
Collaborator

@ipatovanton Disclaimer: we don't officially support that platform.

@webbyhall
Copy link
Author

That's as far as I can scroll so I can't access the submit button when viewed in landscape.

@designbydarren
Copy link

This is also evident on an iPhone 4 using Google Chrome 29.0.1547.11 and Safari (on iOS 6.1.3)

@imawuman
Copy link

in version 3.0.0-rc2, there was a class named "navbar-collapse-scrollable" or "nav-collapse-scrollable" that allowed you to make the collapsed navbar scrollable. I believe this commit #9403 removed that class and documentation

@sergeyd108
Copy link

Have the same problem on android 4.0.4 default browser.

@mdo mdo closed this as completed in eed53d7 Oct 21, 2013
@mdo
Copy link
Member

mdo commented Oct 21, 2013

This should be better now, but not fully fixed. I switched to using overflow-y: auto and tested on my Android running 4.x in the stock browser. Also tested in iOS 7, Chrome, and IE8—all looks good. Scrollbar only appears when the max-height is passed.

For those running into problems with the horizontal mode, the only way to really fix this is to change the max-height on the .navbar-collapse from the current 340px to something like 270px (for one of the most common resolutions). Given that the Android market is saturated with inconsistent sizes, there's no way via CSS to solve this.

Perhaps @fat can revisit the JS in 3.0.2 to detect viewport dimensions and size accordingly, but that sounds bad as well.

@mdo mdo reopened this Oct 21, 2013
@cvrebert
Copy link
Collaborator

If only the Android Browser supported the vh unit...

@damphat
Copy link

damphat commented Nov 8, 2013

Using "overflow-y: auto" make another bug on my Laptop with Google Chrome 30.0

Bug reproduction:

  • Create simple navbar which contain a dropdown menu
  • Shrink browser to make navbar collapsed
  • Open menu (and do not close it, just do the next step)
  • Maximize browser
  • Click dropdown menu
    => the dropdown-menu will not be shown

I have not a phone now for testing but I think this code may fix the problem for both phone and desktop browsers:

//
// navbar.less
//

&.in {
    overflow-y: scroll;
}

@media (min-width: @grid-float-breakpoint) {
    &.in {
        overflow-y: visible;
    }
}

Anybody help me test this?

@mdo
Copy link
Member

mdo commented Dec 14, 2013

Closing this out since I don't see any additional fix for this in the near future. Anything we do will likely have to come with v4 when we can break backward compatibility.

@mdo mdo closed this as completed Dec 14, 2013
stempler pushed a commit to stempler/bootstrap that referenced this issue Apr 11, 2014
…stead of visible to better enable scrolling on Android 4.x
stempler pushed a commit to stempler/bootstrap that referenced this issue Nov 4, 2014
…stead of visible to better enable scrolling on Android 4.x
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

8 participants