Supprimer une action de modération

Salut les agrumes,

En l’état actuel des choses, les staffeux ou admins peuvent ajouter des remarques sur des utilisateurs, mais ne peuvent pas en retirer. Cette fonctionnalité serait bien pour nettoyer un peu de vielles remarques ou des blagues entre staffeux (Arius en aurait bavé là dessus ).

J’ai donc commencé l’implémentation de cette fonctionnalité. Il me reste à écrire les tests et créer des règles CSS bien propres (honte à moi, j’écris directement dans la balise HTML). Mais déjà, j’aimerai votre avis sur le rendu.

Merci d’avance pour vos retours. Ah et au fait, Django ça roxxe !

On m’a appelé ?

(Arius en aurait bavé là dessus ).

Hé, me massacrez pas le profil avec vos remarques ! J’suis sûr que certains s’en sont donné à coeur joie

Edit : pas mal du tout (+ retours du VDD que j’approuve)

Alors je vais faire comme pour les MPs. Même si je me prends des erreurs de NoReverseMatch. Que je ne comprends pas.

EDIT : ah bah c’est bon, un problème dans l’url.

On y a pas touché depuis 2016, on est innocent.

Voilà le nouveau résultat.

Ne reste plus qu’à centrer les checkboxes.

Merci à toi.

Alors le deuxième point c’est bon, je l’ai fait. Par contre le premier je n’y ai pas pensé. Merci de m’avoir soulevé ce point.

Du coup, modification effectuée mais question d’UX : pour les actions de Ban, est-ce qu’il vaut mieux afficher une checkbox grisée et uncheckable ou ne rien afficher du tout ? Je pense personnellement pour la deuxième option.

EDIT : voilà ce que ça rend, pour mieux visualiser. Oh, ne prêtez pas attention à la valeur du karma à gauche, les données sont faussées parce qu’au début je faisais += au lieu de -=.

J’ai une préférence pour une case à cocher grisée. Quand l’élément est absent, on peut se demander pourquoi alors qu’en voyant une action qui n’est pas possible, on comprend qu’elle est dans ce cas interdite.

J’ai une préférence pour pas de case à cocher. Une case à cocher désactivée m’indique un élément qui pourrait être ’cochable’ dans d’autres circonstances, et me fera chercher comment l’activer ou ce que j’ai fait de faux pour la rendre inactive. Si un truc ne peut en aucun cas être checké, faut pas mettre de checkbox.

Je suis partisan de l’absence de case également. Par contre, il faut penser au cas où il n’y a que des sanctions et éviter de se retrouver avec une colonne complètement vide. On pourrait mettre un tiret à la place de la checkbox quand il n’y en a pas ou vérifier la présence de notes dans le tableau avant d’afficher la colonne.

Autre chose : l’emplacement du bouton « Supprimer » me paraît peu intuitif. J’opterais bien pour un bouton en haut et/ou en bas du tableau qui apparaît quand on a sélectionné au moins une note.

Le remplacement des cases à cocher désactivées par des tirets est selon moi un bon compromis  !

J’ai décidé, dans le cas des sanctions, d’afficher le caractère — ,comme ça pas de cas particulier à regarder si y’a des bans ou pas.

Pour la position du bouton de suppression, je me suis inspiré de celui des MP, mais on peut très bien le bouger si vous pensez que c’est mieux.

Je suis partisan de l’uniformité des éléments donc j’aime bien ton approche.

Il est à mon avis aussi mal placé dans les MPs, mais je préfère aussi que soit tu changes les deux soit tu mets celui-ci comme les MPs plutôt que rendre le truc incohérent.

Je pense procéder par étape. D’abord je vais terminer ce développement et ensuite je ferai un PR pour modifier la place des deux bouttons supprimer.

Par contre, je vais avoir besoin d’aide pour les tests. Je n’ai aucune idée de comment tester ça.

Je remonte ma demande d’aide. Je ne vois pas par quel bout commencer. Je dois créer un profil avec profile = ProfileFactory() et ajouter puis supprimer des remarques, c’est ça ?

Le mieux est que tu viennes sur irc

Je veux bien le canal s’il-te-plaît.

#zds-dev sur freenode.