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

Factorisation du theme "defaut" #267

Closed

Conversation

bazooka07
Copy link
Collaborator

  • Le fichier articles-loop.php contient la boucle sur les articles
    commune à home.php, categories.php, tags.php et archives.php.
  • Une constante FULL_WIDTH est définie dans articles-full-width.php et
    static-full-width.php pour utiliser article.php et static.php sans
    sidebar.
  • header.php et footer.php regroupent toutes les parties communes aux
    fichiers.
  • { text-align: center; } pour la tags-list.

* Le fichier articles-loop.php contient la boucle sur les articles
communes a home.php, categories.php, tags.php et archives.php.
* Une constante FULL_WIDTH est definie dans articles-full-width.php et
static-full-width.php pour utiliser article.php et static.php sans
sidebar.
* header.php et footer.php regroupent toutes les parties communes aux
fichiers.
* { text-align: center; } pour la tags-list.
@pluxml
Copy link
Collaborator

pluxml commented Dec 8, 2017

Pas de factorisation du theme par défaut, car il n'est plus possible de donner un comportement différent en fonction de la nature des pages. Ce n'est pas l'objectif de ce theme qui se veut à la fois être didactique et être un squelette de base pour personnaliser son thème.
Je ne valide pas ce PR

@pluxml pluxml closed this Dec 8, 2017
@bazooka07
Copy link
Collaborator Author

bazooka07 commented Dec 8, 2017

Par expérience, répéter plusieurs fois une portion de code dans un programme informatique est une source d'ennuis.
Si on veut faire une modification à cette portion de code, pour corriger un bug ou ajouter une fonctionnalité, a minima la modification sera à faire à chaque copie et ce sera une perte de perte.
Pire, si la modification ne répond pas aux attentes ou s'avère plus compliquée, il faut corriger le même code à plusieurs endroits, et peut-être plusieurs fois.

categories.php, tags.php et archives.php ne sont que des filtres appliqués à la liste d'articles avant appel de la boucle d'articles contenu dans home.php.

Si certains veulent personnaliser la présentation (chapô) des articles, les modifs ne seront à faire que dans articles-loop.php.
Dans le cas où on souhaite une présentation spéciale pour les catégories, tags ou archives, il sera toujours temps de remplacer include('articles-loop.php') par le contenu du fichier articles-loop.php.

Je ne trouve pas très didactique d'inciter les gens à dupliquer leurs codes à tout va. Il est plutôt souhaitable d'inciter à modulariser le code par des include, des fonctions ou des class.

Concernant article-full-width.php et static-full-width.php, il manque cruellement une option à l'édition des articles et des pages statiques pour afficher ou non la sidebar. Cela éviterait une nouvelle fois à dupliquer du code dans le thème "defaut" ou personnalisé.

A réfléchir

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.

None yet

2 participants