NextINpact / Next

18 stars 0 forks source link

[Suggestion technique] Support multi-thème #198

Open francois-dorin opened 12 months ago

francois-dorin commented 12 months ago

Bonjour à l'équipe !

En regardant d'un peu plus près le code pour mettre à jour mon extension (pour être précis, les fichiers .sass), il m'est venue une suggestion pour vous pour la gestion des thèmes.

La solution actuellement retenue est fonctionnelle (ce n'est pas du tout le souci). Par contre, je pense qu'elle risque d'être difficile à maintenir à plus ou moins long terme, en fonction de vos choix (notamment si vous souhaitez rajouter encore des thèmes supplémentaires à l'avenir).

En effet, pour chaque thème que vous ajoutez, vous aller devoir modifier la quasi-totalité des fichiers .sass. Et à chaque fois que vous aller rajouter des éléments, les modifier, etc. au niveau du site, il va falloir rajouter des règles pour chacuns des thèmes existants et tester ensuite l'intégralité des thèmes. Ce n'est pas infaisable, mais cela risque de demander pas mal de maintenance.

Ma suggestion : utiliser des variables CSS. C'est l'approche que j'ai choisi pour mon extension.

Grosso modo, voici ce que j'ai fais :

Ainsi, la définition d'un thème, c'est juste quelques lignes.

Voilà, ce n'est qu'une suggestion d'ordre technique, qui va dépendre bien évidemment de vos besoins ;) Et il faudra l'adapter. J'ai du tordre un peu le concept, dans la mesure où je ne peux pas venir modifier directement les sources. L'idéal étant, bien évidemment, de ne pas initialisé des variables SASS avec des variables CSS, mais de mettre d'utiliser directement des variables CSS, et de supprimer les règles qui deviendraient de facto superflu (effet de bord : cela diminue en plus la taille des CSS, le nombre de règles et leur compléxite ;) )

Courage à vous pour la suite !