-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Update bootstrap to 2.2.2 (our 2.2.2.0) #270
Conversation
trisweb
commented
Dec 11, 2012
- From update to match v2.2.2 jlong/sass-bootstrap-defunct#44
- Fully scripted to make future updates yet easier.
- Version numbers bumped to 2.2.2.0
Now automatically takes care of path changes for the includes in bootstrap.scss and responsive.scss (From twbs#258)
Thank you Tristan! This speed is awesome 🚀 |
@thomas-mcdonald any ETA on getting this upgrade out? |
Thanks @trisweb 🍔 |
Thanks! This fixes a bug with dropdown links for mobile. |
Yeah! I'm also waiting for the dropdown fix. Hope this commit can be pulled in soon. |
In the meantime you can replace .../gems/bootstrap-sass-x.x/vendor/assets/javascripts/bootstrap-dropdown.js manually with this: |
Ya it sucks to not work on touch devices. Thanks for the fix! |
|
please merge. thanks |
Will it arrive before Christmas? C'mon |
🚢 |
Update bootstrap to 2.2.2 (our 2.2.2.0)
Thanks! |
Weird, I was hoping this would fix our dropdown mobile bug as well, but it didn't seem to work. I made sure I had the latest:
And our nav code
This is on coinbase.com if anyone has any ideas (can try it in a shrunk down chrome window). Thanks! |
Fixed for now with css so it is always open on mobile: twbs/bootstrap#4550 (comment) But still open to ideas if people have any. Thanks! |
@barmstrong - I had the same problem. 2.2.2 did NOT fix dropdowns on mobile. It's really this pull that will fix it if merged: twbs/bootstrap#6488 Here's a temporary workaround that worked for me. Place this in your JS after the bootstrap JS is loaded: // Temp Fix: Remove after Bootstrap does the right thing and removes touchstart (see issue https://github.com/twitter/bootstrap/issues/6488)
$('a.dropdown-toggle, .dropdown-menu, .dropdown-menu a, .dropdown-menu .dropdown-submenu a').on('touchstart.dropdown.data-api', function (e) {
e.stopPropagation();
}); Once a fix is (hopefully) in the next version, we'll pull it in from upstream. |
@barmstrong correction, I'm doing some more research. 2.2.2 did not appear to fix anything for me, though this line from this pull should be the fix: https://github.com/thomas-mcdonald/bootstrap-sass/pull/270/files#L8R157 |
I'm using version 3.3.3 of this gem and it is still not fixed. Am I missing something? |