Re: [html5up_editorial] 2 commits

C’est donc un bug de scssphp mais qui en fait était déjà là avant et produisait un selecteur css invalide, et qui maintenant, se controlant lui même, nous jette à la face sa propre erreur :p

https://github.com/scssphp/scssphp/issues/273

--
Cédric
Le 16 nov. 2020 à 14:52 +0100, Matthieu Marcillaud <[email protected]>, a écrit :
> Le 16/11/2020 à 13:32, [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...
>
> Passer sur irc avant, ou poser la question sur .devel, aurait pu aider
> je pense.
>
> Cette nouvelle version de la lib est plus proche de la spec de Sass sur
> les opérateurs de calculs, qui nécessitent un espace de part et d’autre
> de l’opérateur. Je suppose que le problème est là ? Ou pas
>
> Et s’il n’est pas là, tu aurais pu en parler avant de virer tous tes
> fichiers :)
>
> Dans le cas présent, si tu pointes
> https://git.spip.net/spip-contrib-squelettes/html5up_editorial/src/commit/c9443814608606e66be868230790564040d1d278/css/libs/_skel.scss#L257
> et les lignes suivantes, il y a de quoi peut être être sur des cas
> limites pour le parseur...
>
> $x est définit à par exemple `\28xlarge\29`
> Et utilisé comme sélecteur après : `.\31 2u#{$x}`
>
> - Possiblement les \ ne sont plus autorisés dans les sélecteurs ?
> - Possiblement un sélecteur ne peut commencer par un chiffre ?
> - Possiblement un bug laisse ce `$` trainer ? mais pourquoi sur le 39 et
> pas les autres ?...
>
> Je n’ai pas la réponse là, mais c’est peut être des choses comme ça à
> regarder...
>
> MM.
>
> _______________________________________________
> liste: https://listes.rezo.net/mailman/listinfo/spip-dev
> doc: https://www.spip.net/
> dev: https://core.spip.net/
> irc://irc.freenode.net/spip