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

Navbar dropdown on iOS #14603

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
4 participants
@jesperkc
Copy link
Contributor

commented Sep 12, 2014

In the navbar, open one dropdown.
After open another, without closing the first.
This will cause the 'dropdown-menu' to be cut off as you can see here:
http://jsbin.com/jeyuwi/1

Removing .translate3d(0, 0, 0); seems to fix this.

Navbar dropdown on iPad
In the navbar, open one dropdown.
After open another, without closing the first.
This will cause the 'dropdown-menu' to be cut off as you can see here:
http://jsbin.com/jeyuwi/1

Removing .translate3d(0, 0, 0); seems to fix this.

@cvrebert cvrebert added the css label Sep 12, 2014

@cvrebert

This comment has been minimized.

Copy link
Member

commented Sep 12, 2014

Not seeing any visual bug with Safari v7.0.6, Firefox, or Chrome, all on OS X.
I think @mdo has an iPad for testing though.

@hnrch02 hnrch02 added the confirmed label Sep 12, 2014

@hnrch02 hnrch02 added this to the v3.2.1 milestone Sep 12, 2014

@hnrch02

This comment has been minimized.

Copy link
Member

commented Sep 12, 2014

I can confirm this using Safari on my iPad running iOS 7.1.2.
Safari 7.0.6, Chrome 38.0.2125.58 beta, Firefox 32.0.1 and Opera 24.0.1558.53 on OS X 10.9.4 are not affected.

@hnrch02

This comment has been minimized.

Copy link
Member

commented Sep 12, 2014

Related to #14227. translate3d was introduced by #13649.

@cvrebert

This comment has been minimized.

Copy link
Member

commented Sep 12, 2014

I can reproduce on my iPhone 5S running iOS 7.0.4.

@cvrebert cvrebert changed the title Navbar dropdown on iPad Navbar dropdown on iOS Sep 12, 2014

@hnrch02

This comment has been minimized.

Copy link
Member

commented Sep 12, 2014

Interestingly, I can't reproduce on my iPhone 5 also running iOS 7.1.2. @cvrebert Any chance you could update to >=7.1?

@hnrch02

This comment has been minimized.

Copy link
Member

commented Sep 12, 2014

I'm also unable to reproduce in the iOS Simulator 7.1. Retesting now on my iPad also makes me unable to reproduce. I'm only able to reproduce on my iPad when the view is zoomed in significantly.

@hnrch02

This comment has been minimized.

Copy link
Member

commented Sep 22, 2014

I was finally able to reproduce this in the iOS Simulator 8. Here's a screencap:
Screencap

mdo added a commit that referenced this pull request Sep 22, 2014

Fixes #14227, #14603: Drop translate3d for translate on navbars to av…
…oid iOS rendering problems and ensure a more predictable base styling
@mdo

This comment has been minimized.

Copy link
Member

commented Sep 22, 2014

Fixed in #14667.

@mdo mdo closed this Sep 22, 2014

@cvrebert

This comment has been minimized.

Copy link
Member

commented Oct 3, 2014

We should file an iOS WebKit bug about this.

@cvrebert cvrebert reopened this Oct 3, 2014

MatrixZ pushed a commit to MatrixZ/bootstrap that referenced this pull request Oct 9, 2014

Fixes twbs#14227, twbs#14603: Drop translate3d for translate on navba…
…rs to avoid iOS rendering problems and ensure a more predictable base styling
@hnrch02

This comment has been minimized.

Copy link
Member

commented Oct 23, 2014

@cvrebert You wanna take care of filing that bug report?

@hnrch02 hnrch02 removed this from the v3.3.0 milestone Oct 23, 2014

@cvrebert

This comment has been minimized.

Copy link
Member

commented Oct 29, 2014

@cvrebert

This comment has been minimized.

Copy link
Member

commented Oct 29, 2014

cvrebert added a commit that referenced this pull request Oct 29, 2014

@cvrebert

This comment has been minimized.

Copy link
Member

commented Oct 29, 2014

@hnrch02 Would appreciate if you could join the CC list of https://bugs.webkit.org/show_bug.cgi?id=138162

@hnrch02

This comment has been minimized.

Copy link
Member

commented Oct 29, 2014

Done.

stempler pushed a commit to stempler/bootstrap that referenced this pull request Nov 4, 2014

(less) Fixes twbs#14227, twbs#14603: Drop translate3d for translate o…
…n navbars to avoid iOS rendering problems and ensure a more predictable base styling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.