Confus par « Approve » et « Reject » dans la file d'examen

Continuing the discussion from Feedback on the new Review Queue:

Am I the only person who often gets confused by the “Approve” and “Reject” buttons in the review queue? I do not get alot of flagged users and posts in my community, thank goodness, but that means that I am a bit unfamiliar with the flagging system and terminology as it was in the past and still today don’t have to interact with it very often. I also am a self-diagnosed slysdexic, for sure. As a result,when on that interface my mouse cursor or finger often hovers indecisively over the two buttons not being sure which one to choose.

I have to remember to tell myself the buttons mean:

  • Approve (keep flagged user or post)
  • Reject (delete user or post)

The buttons do NOT mean:

  • Approve (delete user or post)
  • Reject (keep user or post)

In case I am not alone in my confusion here, I’d suggest including text like the above on the button itself or at least in mouseover helper text. Perhaps there could even be a short para somewhere on the review queue page explaining how to use it, or a link to a guide here on meta that explains how moderators are expected to use the review queue.

I would put myself in the same group here. I don’t regularly interact with the flagging system, so it feels a little unfamiliar. It isn’t 100% clear what the buttons will do when you click them.

Once you click on the Agree or Delete buttons, you do get some more detailed descriptions of what will happen.

I like this idea for the disagree/ignore buttons, which do not have a dropdown.

And to make matters worse, the icons mean different things in different contexts. I’ve been bitten by this before:

Thanks, David! I like that dropdown very much.

I guess I am looking mostly at flagged users. In this case you hit APPROVE or REJECT and the user is just instantly approved or deleted, apparently. There is no drop down, and I also wonder what’s actually happened.

If I hit REJECT, does it just delete the user or does it also block the email and IP as I can choose on the user admin screen?

I feel like the drop down on flags is fairly clear (although if anyone has a suggestion for making them more clear I would be interested in hearing what you have to say.)

For reviewing users, I’m wondering if it makes sense to change from:

:+1: Approve :-1: Reject

To:

:heavy_check_mark: Grant Access :x: Delete User…

Where Delete User would pop up to say “Delete the User” and “Delete and block”

Would that be more clear?

Thanks, Robin! Glad to know I am not going crazy on my own and that you think changing this is a good idea.

Your new labels look good to me and are much more clear. I am also looking forward to the popup to delete the user or delete and block. :rocket:

One note - we still have “Approved” and “approved by..” in various places in the UX, like on the user admin settings. Maybe you want to make the connection clearer?

That’s a good point. I think we have to keep the word “Approve” to make sense with other terminology. Here’s what I went with:

https://github.com/discourse/discourse/commit/14f9d40e486c39e74abfacdebbda86db1a8afd70

Sweet! I just tried this to delete some spam users and it’s super clear and easy. Man, this is light years better than what we had before.

Thank you again for taking on this mondo project and always looking out for us. :rocket:

Je viens de traiter un sujet signalé et j’ai à nouveau été confus quant au choix à faire dans cette interface. Il n’y a aucune indication avant de cliquer sur ce qui se passera, et je ne savais pas quoi sélectionner.

Lorsque j’ai sélectionné APPROUVER (ou était-ce ACCORDER ? J’ai oublié), un menu déroulant s’est affiché, semblant indiquer que j’étais d’accord pour dire qu’il s’agissait de spam et qu’il devait être masqué. J’ai ensuite sélectionné l’autre option (DÉSACCORD ?), et une action a été effectuée immédiatement sans menu déroulant. Le message n’était plus signalé et était actif dans le forum. Comme il s’est avéré, c’était bien l’action que je souhaitais, et j’ai ensuite transformé le sujet en message privé pour en discuter avec l’auteur.

Je dois être d’accord (sans jeu de mots) avec ce sujet. Les options « D’accord »/« Pas d’accord » me perturbent lorsque je fais occasionnellement une révision. J’ai simplement cliqué sur « Pas d’accord » et je ne suis pas vraiment sûr de ce qui s’est passé. Maintenant, ce message se trouve dans la section « Rejeté », et je ne sais pas vraiment ce que cela signifie non plus (dans les deux cas, la question porte-t-elle sur le message ou sur le signalement lui-même ? « Pas d’accord » → « Rejeté » donne l’impression que le message a été supprimé).

Pour ajouter à ma confusion, j’ai l’impression d’avoir supprimé ce message, ce que je ne voulais pas faire, et je peux le retrouver via la recherche « Rejeté ». Mais je ne vois aucun moyen d’annuler ma décision. De plus, il est indiqué que le message a été supprimé par l’auteur, donc je ne sais pas si cela résulte de mon action ou de quelque chose qu’ils ont fait indépendamment. Peut-être l’ont-ils vraiment fait après le signalement initial, et c’est pourquoi je n’ai aucun moyen de le restaurer. Mais dans ce cas, pourquoi m’a-t-on demandé de le réviser au départ ?

Essayer de s’y retrouver me fait me sentir comme un utilisateur stupide, et bien que je sache que je pourrais résoudre ce problème, je sais aussi que c’est généralement le signe d’une expérience utilisateur qui doit être repensée. Du moins, c’est ainsi que j’aborde les « utilisateurs stupides » sur mon site.

+1 de ma part. L’ensemble « accepter/rejeter/:+1:/:-1: » me semble incohérent et m’oblige vraiment à lire attentivement à la fois le motif du signalement et l’action. Pour prendre un exemple extrême, je préférerais de loin un bouton vert et un bouton rouge pour « conserver ce message » et « supprimer ce message ». Indépendamment du motif du signalement, cela me dirait immédiatement ce qui se produira lorsque je cliquerai sur un bouton.

Je ne reçois pas beaucoup d’utilisateurs ou de publications signalés, mais je suis toujours aussi confus.

Éviter les mots « accepter » / « rejeter » et simplement indiquer l’action (conserver/supprimer) pourrait être mieux, comme l’a suggéré @bartv.

Salut Stéphane, ravi de te voir ici !

J’ai ajouté ce CSS pour mettre en évidence les boutons « destructifs » — cela aide un peu.

button.reject-post,
button.confirm-spam,
button[data-name="Delete..."],
button[data-name="Delete User..."],
button[data-name="Agree..."], {
	color: #fff;
	background: #e86800;

	.d-icon {
		color: #ffe8d5;
	}
}

Excellente idée ! À ce jour, cela me pose encore problème (je comprends bien le défi, cependant). Devons-nous intégrer cela dans le cœur du système ? Soumettre une PR ? :wink:

Cela me rappelle certains boutons de désactivation du son dans les outils de visioconférence. L’icône du bouton de désactivation reflète-t-elle l’état du mutage ou doit-elle indiquer l’action d’appuyer sur le bouton ?! :confused: Je vais enfiler ma :lab_coat:

Légèrement hors sujet : je n’ai jamais compris non plus ce que signifie « Ignorer ». Certains de mes modérateurs pensaient que cela supprimerait le signalement de leur propre file d’attente tout en le maintenant pour les autres modérateurs, mais non.