Re: [html5up_editorial] 2 commits

Hello,

Soit c’est un problème de mixins qui est pas propre et passait par chance dans une ancienne version de scssphp et alors il faudrait corriger le scss, soit c’est un bug de scssphp et alors il faudrait le signaler pour corriger scssphp.

Parce que planquer le bug sous le tapis en virant le scss au profit d’une css c’est pas super cool et le bug va péter dans la tronche de quelqu’un d'autre, non ?

Tu as pu compiler le scss avec un autre outil ?

Pour le détail, la version 1.3.0 de la lib scssphp a introduit la revalidation des selecteurs calculés lors de la compilation. Ie, après compilation on vérifie que le selecteur généré est css-valide.

Il y a eu un bug sur cette fonction, remonté après publication de la version 1.3.0 mais il était lié aux espaces en debut de selecteur et a été corrigé
https://github.com/scssphp/scssphp/issues/245

Cela dit ici j’ai reussi a reproduire le bug avec un cas test simple, et il compile normalement avec SassMeister, c’est donc bien un bug dans ScssPHP.
https://github.com/scssphp/scssphp/issues/273


Mais le principe de l’open-source c’est que si personne ne signale un bug, personne ne le corrigera...

--
Cédric
Le 16 nov. 2020 à 13:32 +0100, [email protected] <[email protected]>, a écrit :
> salut,
> l'erreur telle quelle :
>
> SCSS : Echec compilation fichier plugins/squelettes/html5up_editorial/css/main.scss
> `$\28xlarge\29` is not a valid Selector in `.\39u\28xlarge\29, .\39u$\28xlarge\29`: failed at `$\28xlarge\29` ScssPhp\ScssPhp\Compiler::evalSelectors on line 1, at column 24
>
> issue des mixins qui ne font plus leur boulot dans le fichier css/libs/_skel.scss l257 de le v1.2.3
> ... il me semble...
>
>
> Le 16/11/2020 à 12:11, Cerdic a écrit :
> > Hello Chankalan,
> >
> > peux-tu dire te quoi tu parles concernant les « erreurs scss » ?
> > Quelles erreurs rencontre-tu ?
> >
> >
> > --
> > Cédric
> > Le 16 nov. 2020 à 10:27 +0100, chankalan <[email protected]>, a écrit :
> > > spip-contrib-squelettes/html5up_editorial | 2 commits
> > > -
> > > Par chankalan, le 16 novembre 2020 à 10h12min :
> > >
> > > éviter les erreurs scss (avec scssphp v2.7.0)
> > > en passant par du css (mais en gardant le scss pour les surcharges)
> > >
> > >
> > > *Ajouté*
> > > css/fontawesome-all.min.css
> > > css/main.css
> > > webfonts/fa-brands-400.eot
> > > webfonts/fa-brands-400.svg
> > > webfonts/fa-brands-400.ttf
> > > webfonts/fa-brands-400.woff
> > > webfonts/fa-brands-400.woff2
> > > webfonts/fa-regular-400.eot
> > > webfonts/fa-regular-400.svg
> > > webfonts/fa-regular-400.ttf
> > > webfonts/fa-regular-400.woff
> > > webfonts/fa-regular-400.woff2
> > > webfonts/fa-solid-900.eot
> > > webfonts/fa-solid-900.svg
> > > webfonts/fa-solid-900.ttf
> > > webfonts/fa-solid-900.woff
> > > webfonts/fa-solid-900.woff2
> > > *Supprimé*
> > > css/_font.scss
> > > css/base/_page.scss
> > > css/base/_typography.scss
> > > css/components/_box.scss
> > > css/components/_button.scss
> > > css/components/_features.scss
> > > css/components/_fonts.scss
> > > css/components/_form.scss
> > > css/components/_icon.scss
> > > css/components/_image.scss
> > > css/components/_list.scss
> > > css/components/_mini-posts.scss
> > > css/components/_posts.scss
> > > css/components/_section.scss
> > > css/components/_table.scss
> > > css/fontawesome/font-awesome.min.css
> > > css/fonts/FontAwesome.otf
> > > css/fonts/fontawesome-webfont.eot
> > > css/fonts/fontawesome-webfont.svg
> > > css/fonts/fontawesome-webfont.ttf
> > > css/fonts/fontawesome-webfont.woff
> > > css/fonts/fontawesome-webfont.woff2
> > > css/fonts/opensans-italic-webfont.woff
> > > css/fonts/opensans-italic-webfont.woff2
> > > css/fonts/opensans-regular-webfont.woff
> > > css/fonts/opensans-regular-webfont.woff2
> > > css/fonts/opensans-semibold-webfont.woff
> > > css/fonts/opensans-semibold-webfont.woff2
> > > css/fonts/opensans-semibolditalic-webfont.woff
> > > css/fonts/opensans-semibolditalic-webfont.woff2
> > > css/fonts/robotoslab-bold-webfont.woff
> > > css/fonts/robotoslab-bold-webfont.woff2
> > > css/fonts/robotoslab-regular-webfont.woff
> > > css/fonts/robotoslab-regular-webfont.woff2
> > > css/layout/_banner.scss
> > > css/layout/_footer.scss
> > > css/layout/_header.scss
> > > css/layout/_main.scss
> > > css/layout/_menu.scss
> > > css/layout/_sidebar.scss
> > > css/layout/_wrapper.scss
> > > css/main.scss
> > > css/theme.scss
> > > *Modifié*
> > > css/libs/_skel.scss
> > > css/spip.scss
> > > inclure/head.html
> > >
> > > Détails : https://git.spip.net/spip-contrib-squelettes/html5up_editorial/commit/7167ea9f7e993fc7e9169f30e45f5914a023de6a
> > >
> > > ==============================
> > > Par chankalan, le 16 novembre 2020 à 10h25min :
> > >
> > > v1.3.0 toujours en test
> > >
> > >
> > > *Modifié*
> > > paquet.xml
> > >
> > > Détails : https://git.spip.net/spip-contrib-squelettes/html5up_editorial/commit/184bad2c7894bd2d24d1ee4dbdb7746dd93dca12
> > >
> > > _______________________________________________
> > > [email protected] - https://listes.rezo.net/mailman/listinfo/spip-zone-commit
>
> --
> ----
> chan