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

Scss version #19

Open
wants to merge 15 commits into
from

Conversation

Projects
None yet
3 participants
@mistergraphx

Bonjour,

Je propose une version scss/sass du travail effectué sur TinyTypo.
Less étant de moins en moins utilisé, y compris par les frameworks lourds (bootstrap v4, Fundation, ..), je pense qu'il est judicieux de proposer aux intégrateurs un solution plus pérenne.
J'utilise scss pour ma pars depuis 2 ans maintenant, je me propose donc de maintenir cette version ou branche ...

Bonne journée

@tetue

This comment has been minimized.

Show comment
Hide comment
@tetue

tetue Dec 23, 2015

Owner

Yeees ! Avec plaisir !

Owner

tetue commented Dec 23, 2015

Yeees ! Avec plaisir !

@mistergraphx

This comment has been minimized.

Show comment
Hide comment
@mistergraphx

mistergraphx Dec 23, 2015

;-) ha ben j'm attendais pas à une réponse si rapide ^^ c'est la magie du web !
j'ai un autre projet en cours que j'ai bêtement appellé SassySpip, dont le but est de proposer un "Kit" scss basé sur la dist et les styles de base : ceci est un portage des travaux que tu as éffectué sur la structuration des classes de spip. Bref, tinytypo pourrait ou devrais être un module de ce Kit/framework spipien dans l'idée …

;-) ha ben j'm attendais pas à une réponse si rapide ^^ c'est la magie du web !
j'ai un autre projet en cours que j'ai bêtement appellé SassySpip, dont le but est de proposer un "Kit" scss basé sur la dist et les styles de base : ceci est un portage des travaux que tu as éffectué sur la structuration des classes de spip. Bref, tinytypo pourrait ou devrais être un module de ce Kit/framework spipien dans l'idée …

@mistergraphx

This comment has been minimized.

Show comment
Hide comment
@mistergraphx

mistergraphx Dec 23, 2015

il faudrait déterminer le niveau de spécification scss que l'on souhaite :
si c'est a partir de scssPhp, Ruby Sass/compass, libSass ... dans l'idée à savoir si on veux utiliser avec le plugin scssphp de spip, on est limité a certaines fonctions du langage.

il faudrait déterminer le niveau de spécification scss que l'on souhaite :
si c'est a partir de scssPhp, Ruby Sass/compass, libSass ... dans l'idée à savoir si on veux utiliser avec le plugin scssphp de spip, on est limité a certaines fonctions du langage.

@rastapopougros

This comment has been minimized.

Show comment
Hide comment
@rastapopougros

rastapopougros Dec 24, 2015

Je plussoie aussi, merci. :)
J'avais commencé à le faire rapidement en local et finalement j'étais passé à autre chose, pas eu le temps.

Pour ce qui est de SCSS et SPIP, moi j'utilise SCSSPHP fourni dans le plugin idoine. Du coup ça limite en version car ça ne gère pas tout. Mais bon pour la majorité des choses ça suffit.

J'utillise TinyTypo comme base l'un des mini-frameworks de base de mon échafaudage IntéGraal ici :
http://zone.spip.org/trac/spip-zone/browser/_squelettes_/integraal/theme/css

Mais du coup pour l'instant je l'intègre tel quel en entier, sans variable sans rien, les surcharges se faisant en tant que surcharge CSS dans les feuilles ensuite.

Je plussoie aussi, merci. :)
J'avais commencé à le faire rapidement en local et finalement j'étais passé à autre chose, pas eu le temps.

Pour ce qui est de SCSS et SPIP, moi j'utilise SCSSPHP fourni dans le plugin idoine. Du coup ça limite en version car ça ne gère pas tout. Mais bon pour la majorité des choses ça suffit.

J'utillise TinyTypo comme base l'un des mini-frameworks de base de mon échafaudage IntéGraal ici :
http://zone.spip.org/trac/spip-zone/browser/_squelettes_/integraal/theme/css

Mais du coup pour l'instant je l'intègre tel quel en entier, sans variable sans rien, les surcharges se faisant en tant que surcharge CSS dans les feuilles ensuite.

@mistergraphx

This comment has been minimized.

Show comment
Hide comment
@mistergraphx

mistergraphx Dec 24, 2015

Yes, c'est pour ça que je pose la question, j'avais vu passé les commit de inteGraall (j'utilise quasiement la même base en fait...), je compile avec Gulp et Node-sass/Lib Sass pour ma part, mais pour ce qui est de TinyTipo on as pas besoin non plus de sortir l'artillerie lourde, j'imagine que l'ambition n'est pas d'en faire une usine a gaz et de rester léger... Sinon autant utiliser TypePlate ou une autre Lib dédié (ce que je fais suivant les projets ...). Donc on reste sur une compat scssphp.
Peut être améliorer la gestion des rythmes verticaux et des tailles de titre qui sont en dur actuellement, me paraitrait bien ... du coup vu que tu utilise Bourbon/neat on peut imaginer partager les mêmes variables pour ce qui est des rytmes et autres... séparer vers un dossier base tout ce qui est html tags, serait bien ptet, ... a voir et discutter

Yes, c'est pour ça que je pose la question, j'avais vu passé les commit de inteGraall (j'utilise quasiement la même base en fait...), je compile avec Gulp et Node-sass/Lib Sass pour ma part, mais pour ce qui est de TinyTipo on as pas besoin non plus de sortir l'artillerie lourde, j'imagine que l'ambition n'est pas d'en faire une usine a gaz et de rester léger... Sinon autant utiliser TypePlate ou une autre Lib dédié (ce que je fais suivant les projets ...). Donc on reste sur une compat scssphp.
Peut être améliorer la gestion des rythmes verticaux et des tailles de titre qui sont en dur actuellement, me paraitrait bien ... du coup vu que tu utilise Bourbon/neat on peut imaginer partager les mêmes variables pour ce qui est des rytmes et autres... séparer vers un dossier base tout ce qui est html tags, serait bien ptet, ... a voir et discutter

@mistergraphx

This comment has been minimized.

Show comment
Hide comment
@mistergraphx

mistergraphx Dec 24, 2015

Et en fait ne faut il pas faire une branche scss et une less ? ce qui éviterait que le dev qui utilise l'un ou l'autre n'embarque pas la totale ...

Et en fait ne faut il pas faire une branche scss et une less ? ce qui éviterait que le dev qui utilise l'un ou l'autre n'embarque pas la totale ...

mistergraphx added some commits Dec 28, 2015

theme et encodage
Ajout d'un fichier theme, reprenant les styles de la démo du site
tinytypo

Forcer l’encodage dans les fichiers quand il y’a la présence de
caractères unicodes dans les commentaires : a corriger/supprimer et
faire le lien vers le site unicodetable ….
validation html5
On ne met pas de role ARIA quand la definition est déjà sémantique , on
précise le role=«  » quand on utilise une balise générique <div|span\…>
@tetue

This comment has been minimized.

Show comment
Hide comment
@tetue

tetue Apr 7, 2016

Owner

Donc, c'est une très bonne idée.

Question bête : est-ce que les deux sousrep, scss et less, peuvent co-exister au dépôt, sans se gêner ? Sans trop d'incidence sur les feuilles de style générées ? Si oui, j'accepte. Car non, y'a pas plus de raison de faire une branche dédiée SCSS, qu'une branche CSS (pour celleux qui n'utilisent que le CSS de Tiny Typo), ou HTML (pour celleux qui n'utilisent que le HTML)…

Par contre ton PR contient des commits qui ne le rendent pas pas acceptables en l'état :

  • 5c00a91 : invalide le RGAA. En effet, les roles ARIA sont nécessaires.
  • fcfcd4b : tinytypo est une base CSS. En tant que telle, elle ne doit pas contenir de thème, variantes, déclinaisons, etc.

Peux-tu refaire ton PR sans ces commits ?

Owner

tetue commented Apr 7, 2016

Donc, c'est une très bonne idée.

Question bête : est-ce que les deux sousrep, scss et less, peuvent co-exister au dépôt, sans se gêner ? Sans trop d'incidence sur les feuilles de style générées ? Si oui, j'accepte. Car non, y'a pas plus de raison de faire une branche dédiée SCSS, qu'une branche CSS (pour celleux qui n'utilisent que le CSS de Tiny Typo), ou HTML (pour celleux qui n'utilisent que le HTML)…

Par contre ton PR contient des commits qui ne le rendent pas pas acceptables en l'état :

  • 5c00a91 : invalide le RGAA. En effet, les roles ARIA sont nécessaires.
  • fcfcd4b : tinytypo est une base CSS. En tant que telle, elle ne doit pas contenir de thème, variantes, déclinaisons, etc.

Peux-tu refaire ton PR sans ces commits ?

@mistergraphx

This comment has been minimized.

Show comment
Hide comment
@mistergraphx

mistergraphx Apr 9, 2016

Question bête : est-ce que les deux sousrep, scss et less, peuvent co-exister au dépôt, sans se gêner ? Sans trop d'incidence sur les feuilles de style générées ?

je ne pense pas, juste il va falloir se baser sur les css générée et que les deux versions soit synchro pour restituer le même résultat.

Peux-tu refaire ton PR sans ces commits ?

Oui, biensur je revois ça, et je re-publie.

Pour les roles aria sur des tag html5 sémantiques j'ai cru lire que c'est redondant et non recommandé, cela dit toléré comme une compatibilité/support des vieux navigateurs/readers

Cela dit c'est une page de démonstration donc pas vraiment important pour moi, comme le thème c'est juste pratique pour faire un boilerplate on pourrait ne mettre que les classes vides sans déclaration.

Au passage, je ne sais pas si tu as remarqué, mais j'ai commencé documenter le code en utilisant la méthodologie kss, ça me sert à créer des documentations automatiques pour la partie front…

Question bête : est-ce que les deux sousrep, scss et less, peuvent co-exister au dépôt, sans se gêner ? Sans trop d'incidence sur les feuilles de style générées ?

je ne pense pas, juste il va falloir se baser sur les css générée et que les deux versions soit synchro pour restituer le même résultat.

Peux-tu refaire ton PR sans ces commits ?

Oui, biensur je revois ça, et je re-publie.

Pour les roles aria sur des tag html5 sémantiques j'ai cru lire que c'est redondant et non recommandé, cela dit toléré comme une compatibilité/support des vieux navigateurs/readers

Cela dit c'est une page de démonstration donc pas vraiment important pour moi, comme le thème c'est juste pratique pour faire un boilerplate on pourrait ne mettre que les classes vides sans déclaration.

Au passage, je ne sais pas si tu as remarqué, mais j'ai commencé documenter le code en utilisant la méthodologie kss, ça me sert à créer des documentations automatiques pour la partie front…

mistergraphx added some commits Dec 6, 2016

Revert "validation html5"
This reverts commit 5c00a91.
Revert "theme et encodage"
This reverts commit fcfcd4b.
update 1.3.4
- revert des fichiers theme, et de la page test html
- update des dernières motifs du repo
- ajout d’une global version du package pour changer plus facilement la
version des css générées
Ajout de la global $tinytypo-version
de préférence importer les globals avant la doc si on veux avoir le
numero de version dans les css générées
@mistergraphx

This comment has been minimized.

Show comment
Hide comment
@mistergraphx

mistergraphx Dec 6, 2016

Bonjour,

J'ai procédé au modifications que tu m'avais demandé et mis a jour ma branche avec les derniers commits effectués.

Bonjour,

J'ai procédé au modifications que tu m'avais demandé et mis a jour ma branche avec les derniers commits effectués.

@tetue tetue added the enhancement label Sep 2, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment