Convertion de crepbook.cls en deux packages

[Luthaf]

Deuxième tentative !

Fix #17

Tout compile chez moi. Dites-moi si il reste des aberrations visuelles par endroit !

[paternal]

Super @Luthaf. Je mets la relecture de cette branche dans ma toto liste.

[Luthaf]

Avant de merger, il faut que je corrige le problème avec les commandes \and dans la page de titre. Je les ai supprimés un peu violemment de songbook-data, mais c'est une commande LaTeX standard.

Quelqu'un saurait d’ailleurs comment aligner des éléments verticalement ? Ca me fait un peu mal d'utiliser un tableau dans la page de titre.

[paternal]

Avant de merger, il faut que je corrige le problème avec les commandes \and dans la page de titre.

Corrigé : baed32f et patacrep/patadata@21739a7

Je n'ai pas terminé ma relecture : à suivre…

[Luthaf]

Avec ma dernière modification (44a1c4f), cette PR me convient.

Je ne sais pas trop pourquoi, mais je préfère une seule commande TeX a des environnements LaTeX imbriqués de manière étrange. Le rendu actuel est le suivant :

S'il ne vous convient pas (entre autres @crep4ever, on a du un peu changer la mise en page), dites le moi.

[paternal]

Ça me convient.

Par contre, attend avant de merger : je n'ai pas fini ma relecture (jusqu'à présent, je n'ai fait que des relectures de petits morceaux indépendants, mais il faut que je me prenne un moment avec de la disponibilité d'esprit pour faire une relecture globale).

[paternal]

Commentaires :

• chords.sty : Je valide
• tabs.sty : Voir les commentaires dans le commit ba15ad1. Résumé de mes suggestions :
  • quelques scories ;
  • suppression des commandes définies ailleurs.

J'ai l'impression que SongbookUtils.sty concerne des choses nécessaires à la compilation (de songbook-data), et crepbook.sty concerne des choses non nécessaires (des améliorations visuelles principalement). En partant de cette supposition :

• SongbookUtils.sty
  • il faut définir (commentaires) ce que font chacune des options. Même moi qui suis sensé comprendre le code, je ne suis pas sûr de ce qu'elles signifient. Cela sera utile pour : assurer une compatibilité ascendante (être sûr que les versions ultérieures respecteront cela), et pour écrire la documentation (Mettre a jour la documentation #14).
  • quelques commentaires à ba15ad1 et 151f749
• crepbook.sty
  • quelques commentaires à 151f749, 151f749, et 0ef22fa

[paternal]

J'ai fini ma session de relecture. Beaucoup de remarques, mais c'est du bon travail quand même ! :)

[Luthaf]

il faut définir (commentaires) ce que font chacune des options. Même moi qui suis sensé comprendre le code, je ne suis pas sûr de ce qu'elles signifient. Cela sera utile pour : assurer une compatibilité ascendante (être sûr que les versions ultérieures respecteront cela), et pour écrire la documentation (#14).

En pratique, la plupart de options est directement reliée avec une valeur de bookoptions, qui sont déjà définies dans la doc actuelle.

[paternal]

En pratique, la plupart de options est directement reliée avec une valeur de bookoptions, qui sont déjà définies dans la doc actuelle.

D'accord. Il faut que je jette un œil à ça.

[paternal]

À propos des différentes options que je propose d'ajouter pour permettre d'activer/désactiver certaines fonctionnalités dans SongbookUtils.sty et crepbook.sty : je trouve que ces deux paquets apportent de nombreuses modifications, à différents niveaux (tu les avois déjà pas mal regroupées ; j'ai accentué ça en ajoutant encore plus de lignes de % pour les séparer). J'imagine très bien qu'un utilisateur veuille une partie des améliorations et pas toutes.

Par défaut, dans le template default.tex, tout serait activé dans SongbookUtils.sty, et dans le template patacrep.tex, tout seraitactivé dans patacrep.sty.

On peut faire l'inverse, et tout activer par défaut, et ajouter des options pour désactiver certaines parties à la demande.

[Luthaf]

On peut faire l'inverse, et tout activer par défaut, et ajouter des options pour désactiver certaines parties à la demande.

S'il le fait vraiment, je préfère encore ce comportement. Mais pour moi, soit les gens voudront vraiment personnaliser leurs carnets, auquel cas ils utilisent default.tex, et écrivent leur propres modifications en LaTeX, soit ils utilisent la version patacrep.tex, avec la mise en page crepbook, et un rien de changements.

Encore une fois, cela ne me gène pas si il est possible de sélectionner les modifications que l'on veut utiliser, tant qu'elles sont chargées par défaut.

[Luthaf]

le paquet SongbookUtils est chargé dans default.tex
le paquet crepbook est chargé dans patacrep.tex.

Ça me va parfaitement comme fonctionnement.

[paternal]

Je marque l'intégration des résultats de toutes nos discussions à ma todo liste.

[paternal]

J'ai pris en compte tous les commentaires dont nous avions parlé. J'ai compilé tous les carnets (exemples et data), et ça fonctionne.

Si ça vous convient, on peut merger dans next.