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

Upgrade to Bootstrap 4.0.0.alpha6 #1776

Closed
wants to merge 12 commits into from

Conversation

jhawthorn
Copy link
Contributor

@jhawthorn jhawthorn commented Mar 16, 2017

Previously we were on Bootstrap 4.0.0.alpha2. Changes can be seen in their Release notes

Notable to us:

  • col-xs-3 et al. are gone. Replaced with col-3.
  • Flexbox is on by default 👍 (we had already turned on the optional flexbox support)
  • Classes for breadcrumbs changed (easy fix)
  • Some variables and mixins changed

I can't be 100% sure this didn't mess up any styling, but in a quick browse through the admin I didn't spot anything off.

@jhawthorn jhawthorn changed the title Bootstrap 4.0.0.beta6 Upgrade to Bootstrap 4.0.0.beta6 Mar 16, 2017
@jhawthorn jhawthorn changed the title Upgrade to Bootstrap 4.0.0.beta6 Upgrade to Bootstrap 4.0.0.alpha6 Mar 16, 2017
@jhawthorn
Copy link
Contributor Author

Hmm we seem to be running into twbs/bootstrap#21607 so we may have to put off this upgrade

@jhawthorn
Copy link
Contributor Author

jhawthorn commented Mar 16, 2017

I tried setting enable-transitions: false, and it didn't help

EDIT: setting animation: false when calling $.fn.tooltip() fixes this.

Previously normalize.css set table's border-spacing to 0, bootstrap now
uses reboot.css, which doesn't reset this property.
Bootstrap has removed col-xs-N in the latest alphas.
This works around a bootstrap issue resulting in

    Error: Tooltip is transitioning

We should be able to re-enable this for the next release if desired.
This works around a bootstrap 4.0.0.alpha6 issue resulting in

    Error: Tooltip is transitioning

We should be able to re-enable this for the next release if desired.
@jhawthorn
Copy link
Contributor Author

Closing for #1816

@jhawthorn jhawthorn closed this Apr 3, 2017
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

Successfully merging this pull request may close these issues.

1 participant