Skip to content
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

Modularisation et templating #14 #19 #20

Merged
merged 15 commits into from
Mar 30, 2022
Merged

Modularisation et templating #14 #19 #20

merged 15 commits into from
Mar 30, 2022

Conversation

YannisDelmas
Copy link
Owner

Resolves #19

Resolves #14

@enguerranws
Copy link
Collaborator

Je repasserai bien sur la structure du dossier pour ranger les twig de partials et les distinguer des twig de contenus, tu en dis quoi @YannisDelmas ?

@YannisDelmas
Copy link
Owner Author

Je repasserai bien sur la structure du dossier pour ranger les twig de partials et les distinguer des twig de contenus, tu en dis quoi @YannisDelmas ?

Oui, volontiers. Je me posais la question de déplacer les partials dans assets/twig ou quelque chose comme ça.

Je ne l'ai pas fait parce que j'ai buté sur la question de la table des matières _toc.twig. À terme, il faudrait qu'elle soit intégrée sous forme de menu dans toutes les pages, pour bien faire. Du point de vue du build, c'est donc un partial. Pour autant, d'un point de vue sémantique, c'est plutôt du contenu. Je me prends peut être la tête pour rien… Je te laisse trancher 😄

@enguerranws
Copy link
Collaborator

@YannisDelmas , je te propose qu'on finalise la proposition concernant la structure et qu'on la merge sur cette branche, et qu'on merge tout dans main ensuite ?

@YannisDelmas
Copy link
Owner Author

OK pour moi, je regarde ta proposition.

enguerranws and others added 5 commits March 28, 2022 11:35
`experimental/grunt structure`: tout semble OK, j'ai rectifié les composants de JS et PHP (pour l'instant vides). La compilation donne exactement le résultat antérieur. Je valide, donc.
@YannisDelmas
Copy link
Owner Author

@enguerranws J'ai relu les documents produits par cette branche. Ça a l'air de fonctionner sans accroc. Les fichiers HTML produits sont identiques à l'octet près, apparemment. Peux-tu juste vérifier que c'est aussi le cas chez toi et valider la requête merge?

@enguerranws
Copy link
Collaborator

@YannisDelmas C'est bon pour moi !

@YannisDelmas YannisDelmas merged commit 107f989 into main Mar 30, 2022
@YannisDelmas
Copy link
Owner Author

Ça y est j'ai publié sur main et j'ai branché /docs sur la publication https://yannisdelmas.github.io/beau-code-web/. J'ai corrigé aussi un petit oubli dans README.md directement sur main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Templating du projet modularisation
2 participants