-
Notifications
You must be signed in to change notification settings - Fork 4
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
Utilisation de la Gem dsfr-assets pour le guide #202
base: main
Are you sure you want to change the base?
Conversation
gem 'nanoc-live' | ||
gem 'sass-embedded' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pourquoi il faut sass-embedded
? c'est du pure CSS non ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aujourd’hui c’est du scss et on avait le package sass
Après on peut se poser la question de l’intérêt d’utiliser du scss ici
Dans tous les cas, j'ai laissé en draft parce qu’il y a des éléments du DSFR qui n’existent pas dans dsfr-assets
donc il y a encore des petites choses à solutionner
// Import the DSFR | ||
@import "dsfr/dist/dsfr.min"; | ||
@import "dsfr/dist/utility/utility.min"; | ||
@use "dsfr.min"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aaaaah
|
||
o | ||
Nanoc::Filter.define(:dart_sass) do |_, opts| | ||
Sass.compile(opts[:target], style: :compressed).css |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
my bad mais du coup le require d'en haut m'interpelle
Maintenant que nous avons une Gem dsfr-assets, il peut être intéressant de l’utiliser plutôt que d’utiliser le package npm.