Skip to content

[ci skip] Add Bootstrap and Bulma to the CSS processors' list #43254

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

Merged
merged 1 commit into from
Sep 20, 2021

Conversation

alexandreruban
Copy link
Contributor

Closes #43251

Summary

When running rails new --help the bootstrap and bulma options for CSS processors were not documented.
The rails new --css=bootstrap option already works and the rails new --css=bulma option will work in the cssbundling-rails next release thanks to this merged pull request.

@rails-bot rails-bot bot added the railties label Sep 19, 2021
@ceritium
Copy link
Contributor

Just a few thoughts:

  • bootstrap is available on cssbundling-rails@0.1.8, the current rails Gemfile.lock requires cssbundling-rails@0.1.0.
  • bulma is not release yet.
  • Would make sense to expose the list of available frameworks from cssbundling-rails so rails can use it in the help message?

@ceritium
Copy link
Contributor

@alexandreruban FYI #43255

@alexandreruban alexandreruban force-pushed the bootstrap-bulma-css-processors branch from 1c85bb4 to ff6c065 Compare September 20, 2021 08:48
@alexandreruban
Copy link
Contributor Author

@ceritium thank you very much for your comments! I updated the PR according to #43255.

I still think it's worth explicitly mentioning both Bulma and Bootstrap because they are two popular frameworks. A few hours ago, a new version of cssbundling-rails was released, and now the --css=bulma option is working correctly.

@dhh dhh merged commit 0d32bf3 into rails:main Sep 20, 2021
@alexandreruban alexandreruban deleted the bootstrap-bulma-css-processors branch September 20, 2021 09:57
dhh pushed a commit that referenced this pull request Sep 20, 2021
* main:
  [ci skip] Add Bootstrap and Bulma to the CSS processors' list (#43254)
  Add missing configuration to middleware test
  Add ActionDispatch::ServerTiming to the list of middlewares in test
  Add CHANGELOG entry for ServerTiming middleware [ci skip]
  Add Server Timing middleware (#36289)
  Don't overwrite default opts in rich_text_area_tag (#43156)
  Only call `wrap_parameters` if it's defined (#43259)
  Suggest a CSP that's compatible with Turbo + import map (#43227)
  Fix app generator tests
  Fix docs spacing
  Fix typo: integer numbers (not integral)
  Add `beginning_of_week` option to `weekday_options_for_select`
  Pluralize the heading on the index page generated with scaffold
  Move the parameter wrapper code to the ActionController::Railtie class
  Add explicit rendering to DiskController#update
  Fix update & destroy queries when default_scope is nillable with all_queries: true
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bootstrap and Bulma CSS options is not shown at rails new on 7.0.0.alpha2
3 participants