search

patacrep / patanet

Web interface for LaTeX songbook generation
GNU Affero General Public License v3.0
10 stars 3 forks source link

Download songbook #129

Closed oliverpool closed 9 years ago

oliverpool commented 9 years ago

Fix #128

Voici donc une proposition de redesign de la partie "Téléchargements". down Plutôt qu'une liste empilée, j'ai choisi d'afficher les mises-en-pages les unes à côté des autres. Au survol, les options supplémentaires apparaissent (en bas à gauche par exemple). L'état de compilation est mis à jour via AJAX (Compilation en attente/en cours/télécharger) Les images reprennent exactement le "titre" (Portait avec accords) par exemple, et devrait être assez éloquentes.

Si cela vous convient, avant de merger, il faut que je supprime l'attribut de titre, et que j'en profite pour redessiner la page de création d'une mise en page

Luthaf commented 9 years ago

Que de commits ! Je regarderai ça plus tard =)

Sinon, à chaud, je trouve ça plus sympa que la version précédente, mais il manque une information : est-ce que le carnet est en A5 ou en A4. Je ne sais pas trop comment on peut afficher cette information simplement.

oliverpool commented 9 years ago

Ce n'est pas encore implémenté: https://github.com/patacrep/patanet/blob/python3/generator/forms.py#L224

Je pensais l'intégrer en tant qu'information principale: sur la "couverture" (au dessus de la note de musique).

oliverpool commented 9 years ago

Ca fait suffisament de commit pour cette Pull Request: elle est prête à être mergé (hors bug de derrière les fagots)

Luthaf commented 9 years ago

C'est bon pour moi, avec quelques commentaires !

oliverpool commented 9 years ago

Avec beaucoup de commentaires, tu veux dire!

Merci bien d'avoir tout relu: je reposterai un commentaire quand j'aurai tout corrigé :-)

oliverpool commented 9 years ago

Update: J'ai rajouté des commentaires : j'espère qu'ils sont compréhensibles/suffisants. Lors d'une erreur de compilation, un lien invite à utiliser le formulaire de contact (avec sujet pré-rempli). J'ai corrigé des bugs dûs au fait qu'un layout peut avoir plusieurs task.

Si tu valides le tout, tu peux merger!

Luthaf commented 9 years ago

Ya tout bon !

Je regarderai les erreurs d'encoding sur mon VPS la semaine prochaine, histoire d'essayer de déployer tout ça =)

oliverpool commented 9 years ago

Cool: merci pour la relecture!