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

Bootstrap 4.0.0-beta #2156

Merged
merged 4 commits into from
Aug 16, 2017
Merged

Bootstrap 4.0.0-beta #2156

merged 4 commits into from
Aug 16, 2017

Conversation

jhawthorn
Copy link
Contributor

@jhawthorn jhawthorn commented Aug 15, 2017

Bootstrap 4.0.0 has entered beta! Hopefully that means this is the last PR with such significant changes to scss variable naming 😥

This required including autoprefixer-rails as a dependency. We probably should have done this for previous versions, but it is now necessary even for latest browsers.

This required some variable changes. Briefly:

  • s/font-size-h1/h1-font-size/
  • s/$brand-(primary|secondary|success|info|warning|danger|light|dark)/theme-color("\1")/g
  • s/input-border-focus/input-focus-border-color/
  • s/input-bg-disabled/input-disabled-bg/
  • s/input-color-placeholder/input-placeholder-color
  • Use grid-gutter-width instead of grid-gutter-widths
  • Use input-btn-padding and input-btn-line-height

TODO

  • Fix tooltip colouring

Copy link
Contributor

@cbrunsdon cbrunsdon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haven't booted it up and looked at it, but looks reasonable to me.

Variable changes:

s/font-size-h1/h1-font-size/
s/\$brand-\(primary\|secondary\|success\|info\|warning\|danger\|light\|dark\)/theme-color("\1")/g
s/input-border-focus/input-focus-border-color/
s/input-bg-disabled/input-disabled-bg/
s/input-color-placeholder/input-placeholder-color

Use grid-gutter-width instead of grid-gutter-widths

Use input-btn-padding and input-btn-line-height
I don't know why bootstrap uses -btn- for these, as they're used for all
.form-control inputs also.

Also improves select2 custom styling
@jhawthorn
Copy link
Contributor Author

Updated to include the merged #2133

@jhawthorn jhawthorn changed the title [WIP] Bootstrap 4.0.0-beta Bootstrap 4.0.0-beta Aug 15, 2017
@jhawthorn jhawthorn removed the WIP label Aug 15, 2017
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Booted, verified. 🚢 it

@tvdeyen tvdeyen added this to the 2.4.0 milestone Aug 16, 2017
@jhawthorn jhawthorn merged commit 28fda1d into solidusio:master Aug 16, 2017
@davidmaogomezz
Copy link

davidmaogomezz commented Nov 11, 2017

Hi, How can I use bootstrap in my application (frontend)?

@mamhoff
Copy link
Contributor

mamhoff commented Nov 11, 2017

Damagoza: To use bootstrap, you have to customize the frontend. See https://www.youtube.com/watch?v=kFalBLRMgdI for more information.

Github issues are meant to inform about bugs and/or missing features, by the way. For support questions like this, please use our Slack channel (solidusio.slack.com).

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.

5 participants