Confused by "Approve" and "Reject" in the review queue

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:

방금 플래그가 지정된 주제를 처리했는데, 이 인터페이스에서 무엇을 선택해야 할지 또 혼란스러웠습니다. 클릭하기 전에 어떤 일이 발생할지 알려주는 표시가 전혀 없어서 무엇을 선택해야 할지 확신이 서지 않았습니다.

APPROVE(아니면 AGREE였나요? 기억이 나지 않네요)를 선택했을 때, 해당 게시물이 스팸이며 숨겨져야 한다는 데 동의하는 것처럼 보이는 드롭다운 메뉴가 나타났습니다. 그러자 다른 옵션(DISAGREE?)을 선택했는데, 이번에는 드롭다운 메뉴 없이 즉시 조치가 실행되었습니다. 게시물은 더 이상 플래그가 해제된 상태였고, 포럼에서 활성화되었습니다. 결국 제가 원했던 조치였으며, 이후 해당 게시물을 개인 메시지로 변환해 작성자와 논의했습니다.

이 주제에 동의합니다(비유가 아닙니다). 가끔 리뷰를 할 때 동의/반대 옵션이 혼란스럽습니다. 방금 반대를 눌렀는데 무슨 일이 일어난 건지 정확히 모르겠습니다. 이제 그 게시글이 “거절” 아래에 있는데, 이 말도 정확히 무슨 뜻인지 모르겠습니다(두 경우 모두, 질문의 대상이 게시글 자체인지 플래깅 행위 자체인지요? 반대 → 거절이라는 표현은 게시글이 삭제된 것처럼 들립니다).

혼란을 더하는 건, 제가 원치 않았는데 이 게시글을 제가 삭제한 것 같다는 느낌입니다. “거절” 검색에서 찾을 수는 있지만, 제 결정을 되돌리는 방법을 찾을 수 없습니다. 또한 "작성자 의해 게시글 삭제됨"이라고 나와서, 이것이 제 행동의 결과인지 아니면 작성자가 독자적으로 한 것인지도 알 수 없습니다. 아마도 처음 플래그가 걸린 후 작성자가 실제로 삭제한 것일 수도 있고, 그래서 제가 복구할 방법이 없는 것일 수 있습니다. 그렇다면 왜 제가 처음부터 리뷰를 요청받았는지 모르겠습니다.

이 복잡한 과정을 헤쳐 나가는 것은 저를 바보 같은 사용자로 느끼게 합니다. 제가 결국 방법을 찾아낼 수 있다는 것을 알면서도, 이것이 재설계가 필요한 사용자 경험의 문제라는 냄새가 난다는 것도 압니다. 적어도 제 사이트에서 "바보 같은 사용자"들을 대하는 방식은 그렇습니다.

+1입니다. 전체적인 동의/거부/:+1:/:-1: 구조가 일관성이 없어 보이고, 실제로는 플래그 사유와 액션을 모두 주의 깊게 읽어야 해서 불편합니다. 극단적인 예로, '이 게시글 유지’와 '이 게시글 삭제’를 위한 녹색과 빨간색 버튼을 훨씬 선호합니다. 플래그 사유와 관계없이, 이렇게 하면 버튼을 클릭했을 때 어떤 일이 일어나는지 즉시 알 수 있을 테니까요.

플래그된 사용자/게시글을 많이 받는 편은 아니지만, 나도 항상 혼란스러울 때가 있습니다.

@bartv가 제안했듯이, “동의”/"거부"라는 표현을 피하고 단순히 동작(유지/삭제)을 명시하는 것이 더 좋을 것 같습니다.

스테판, 여기에서 만나서 반가워!

‘파괴적’ 버튼을 강조하기 위해 이 CSS를 추가했는데, 조금은 도움이 될 거야.

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;
	}
}

훌륭한 아이디어네요! 지금까지도 이게 나를 혼란스럽게 합니다 (그런데도 그 어려움은 이해합니다). 이걸 코어에 포함해야 할까요? PR을 제출해볼까요? :wink:

영상 회의의 음소거 버튼을 떠올리게 합니다. 음소거 버튼 아이콘은 음소거 상태를 반영해야 할까요, 아니면 버튼을 누르는 동작을 보여줘야 할까요?! :confused: 내 :lab_coat: …

조금 주제에서 벗어난 이야기지만, '무시’라는 옵션의 의미도 항상 이해가 안 됐습니다. 일부 관리자들은 이 옵션이 자기 큐에서는 플래그를 제거하되, 다른 관리자들의 큐에는 그대로 유지되는 줄 알았지만, 실제로는 그렇지 않았습니다.