Produits Entr'ouvert » Corbo

Je trouve aussi le style un peu trop différent, et même sans ça ce n'est pas très agréable. Et puis ces polices toutes petites, je vais pleurer.

Je suis un peu étonné que ce genre de discussion intervienne après le
développement mais j'imagine que la réflexion se construit en la matière.

Oui il est essentiel d'homogénéiser les interfaces autant que possible d'une
brique à l'autre et d'ailleurs le gros boulot fait pour avoir un thème
commun répond à cette exigence. Sinon autant faire du Cap Demat.

Désolé donc Serghei, mais faut faire comme pour les autres briques.

À sa décharge ça fait 1 mois ou 2 que ces interfaces à base de petits blocs existent (je les ai déjà vu), on aurait pu faire une remarque avant.

C'est curieux que ça existe depuis des mois pour un ticket ouvert il y a 14 jours pendant l'eoday, où tout le monde était présent. Et qu'en local, alors que je tourne master, je n'ai pas ça.

Après, si tu parles des gros blocs avec l'alternance de fond gris / bleu-gris, oui, c'est comme ça depuis le début et si je n'ai pas fait de remarque c'est bien parce qu'il y avait pour moi une refonte à faire et que je trouvais plus pratique d'en parler à l'eoday.

Frédéric Péters a écrit :

Après, si tu parles des gros blocs avec l'alternance de fond gris / bleu-gris, oui, c'est comme ça depuis le début et si je n'ai pas fait de remarque c'est bien parce qu'il y avait pour moi une refonte à faire et que je trouvais plus pratique d'en parler à l'eoday.

Oui je parle de ça, j'ai trouvé ça bizarre dès le départ mais je n'ai pas fait de remarque d'où le "on" dans mon précédent commentaire.

Le patch ne s'applique pas chez moi.

Application de  review backoffice homepage and announce management (#12908)
.git/rebase-apply/patch:196: trailing whitespace.
  </div>  
error: le patch a échoué : corbo/static/css/corbo.css:295
error: corbo/static/css/corbo.css : le patch ne s'applique pas
error: le patch a échoué : corbo/templates/corbo/manage.html:1
error: corbo/templates/corbo/manage.html : le patch ne s'applique pas
error: le patch a échoué : corbo/views.py:135
error: corbo/views.py : le patch ne s'applique pas

Pas besoin d'ajouter des icônes d'édition et de suppression directement dans la liste; et côté CSS le style ajouté à "#management ul li" n'est pas nécessaire. Sur la page d'une catégorie, pareil, et l'édition d'une annonce peut avoir lieu dans sa propre page (il me semblait qu'on avait récemment changé ça). "Category: Vacances", on a arrêté de répéter le type dans le titre; par contre il faudrait avoir "Annonces" dans le fil d'ariane dans le bandeau.

Je note dans la capture la disparition de #12653 (après avoir noté la disparition de #12873).

def get_announces_number et def get_subscriptions_number → plutôt appeler ça ..._count.

côté CSS le style ajouté à "#management ul li" n'est pas nécessaire

Il est toujours là. Ainsi qu'un padding latéral ajouté à div#content. Pour vérifier que c'est ok c'est facile, il suffit d'avoir un onglet avec chrono et un onglet avec corbo, passer de l'un à l'autre doit laisser le titre et l'élément de la liste aux mêmes endroits, aux mêmes tailles.

Je ne pense pas que la date de création de la catégorie apporte une info utile.

par contre il faudrait avoir "Annonces" dans le fil d'ariane dans le bandeau.

Toujours d'application. (sur la page d'une catégorie).

Et sur la page d'une annonce le fil d'ariane devrait être Annonces > nom de la catégorie.

Sur la liste des annonces si on veut afficher une info supplémentaire, affichons la date de publication plutôt que la date de modification.

Je préfère employer un fond grisé pour les éléments pas encore publiés mais si on veut utiliser une icône, so be it, mais il faut alors ajouter un peu de padding, là l'icône d'une annonce publiée colle trop au texte.

Et sur la page d'une annonce le fil d'ariane devrait être Annonces > nom de la catégorie.

Toujours d'application.

Sur la liste des annonces si on veut afficher une info supplémentaire, affichons la date de publication plutôt que la date de modification.

Ok maintenant, mais la manière dont c'est aligné n'est pas terrible, ça doit marcher par chance avec certaines largeurs d'écran, a minimal je retirerais le "width: 25%" du "#manement .datetime".

Sur la page d'accueil, gardons uniquement "Nouvelle catégorie", l'ajout des annonces se faisant à partir de la page d'une catégorie. (et aussi, le choix de la catégorie peut alors être retiré).

On a un formatage relativement standard pour le mode "vide", il faudrait l'utiliser.

<div class="big-msg-info">
  {% blocktrans %}
  This site doesn't have any XXX yet. Click on the "New XXX" button in the top
  right of the page to add a first one.
  {% endblocktrans %}
</div>

Autre point que je n'avais pas encore noté, il manque des boutons "annuler" (boites et pages d'ajout ou de modif de catégorie ou d'annonce).

Simplement déjà mettre à jour gadjo ?

Il est à jour:

commit 8bf9dc3b3bd68db8265dcacbc123e769b188cfe0

Il doit y avoir autre chose de cassé chez toi, cf https://doc-publik.entrouvert.com/notes-de-mises-a-jour/backoffice-ete-2016/

ok prenons le ainsi. À l'occasion ce sera pas mal d'avoir des tests sur le /manage/.