stylesheets: merge print rules with the application stylesheet

- Having a separate stylesheet for print rules is not a best practice
  (it makes an extra network request)
- It makes migrating to Sprockets 4 easier
This commit is contained in:
Pierre de La Morinerie 2021-02-23 14:09:01 +01:00
parent e1a450fd6f
commit e51948ab5c
4 changed files with 56 additions and 55 deletions

View file

@ -5,4 +5,3 @@
// = require ./fonts
// = require leaflet
// = require_tree .
// = stub ./print.scss

View file

@ -1,6 +1,7 @@
@import "colors";
@import "fonts";
@media print {
.new-header,
.sub-header,
footer {
@ -61,3 +62,4 @@ th {
.updated-at {
display: none;
}
}

View file

@ -20,7 +20,6 @@
= preload_link_tag(asset_url("Muli-Bold.woff2"))
= stylesheet_link_tag 'application', media: 'all'
= stylesheet_link_tag 'print', media: 'print'
= Gon::Base.render_data(camel_case: true, init: true, nonce: request.content_security_policy_nonce)

View file

@ -10,5 +10,6 @@ Rails.application.config.assets.paths << Rails.root.join('node_modules', '@reach
Rails.application.config.assets.paths << Rails.root.join('node_modules', '@mapbox', 'mapbox-gl-draw', 'dist')
# Precompile additional assets.
# application.js, application.css, and all non-JS/CSS in app/assets folder are already added.
Rails.application.config.assets.precompile += ['print.css']
# application.js, application.css, and all non-JS/CSS in the app/assets
# folder are already added.
# Rails.application.config.assets.precompile += %w( admin.js admin.css )